[haiku-commits] Re: haiku: hrev53378 - in src: system/kernel/vm kits/media

  • From: Axel Dörfler <axeld@xxxxxxxxxxxxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Fri, 16 Aug 2019 16:12:20 +0200

Hi Augustin,

Am 16/08/2019 um 06:31 schrieb waddlesplash:

Then, it may make sense to only dprintf() for userland areas for now (if
only because this would break compatibility otherwise), but for kernel
areas, I would insist on keeping the panic.
[...]
issues, too. Perhaps I should remove the KDEBUG guards now then?

Sorry, I completely misread that part - using dprintf() is perfectly fine here. I somehow thought it would just allow it instead, and only print a warning. But yes, I agree this warning could be printed without KDEBUG turned on.

Then you can also disregard my comment about compatibility; it still breaks compatibility by disallowing this :-)

That's definitely something to keep in mind when running old BeOS applications. Maybe we want to set this flag for all areas of legacy applications.

Bye,
   Axel.

Other related posts: