[haiku-commits] Re: haiku: hrev51274 - src/system/kernel/fs

  • From: Axel Dörfler <axeld@xxxxxxxxxxxxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Tue, 11 Jul 2017 23:49:23 +0200



Am 11. Juli 2017 20:46:24 MESZ schrieb "Jérôme Duval" <jerome.duval@xxxxxxxxx>:

2017-07-11 17:50 GMT+02:00 Jérôme Duval <jerome.duval@xxxxxxxxx>:
Also I'm suggesting to change the Autolocker behaviour to panic on a
NULL
Lockable in the kernel, to detect such issues.

An alternative to this commit seems to be the following, ie don't call
Lock() in the constructor when lockable is NULL.
One can even remove the check in Lock().

We have the same issue in the Be API where you can call BWindow::Lock() even if 
the window is NULL. So that would have to be deprecated, too, then (and our 
sources need to be proofread).

Bye,
   Axel.

Other related posts:

  1. Home
  2. haiku-commits
  3. Archive
  4. 07-2017