[haiku-commits] Re: r34875 - haiku/trunk/src/system/kernel

  • From: Ingo Weinhold <ingo_weinhold@xxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Sun, 03 Jan 2010 23:42:38 +0100

On 2010-01-03 at 19:24:27 [+0100], axeld@xxxxxxxxxxxxxxxx wrote:
> Author: axeld
> Date: 2010-01-03 19:24:27 +0100 (Sun, 03 Jan 2010)
> New Revision: 34875
> Changeset: http://dev.haiku-os.org/changeset/34875/haiku
> 
> Modified:
>    haiku/trunk/src/system/kernel/kernel_daemon.cpp
> Log:
> * This should hopefully fix the broken kernel - this code relied on clearing
>   the next/previous pointers.

Mmh, weird that I didn't run into it.

> There might be more, though.

I sure hope not. It's rather ugly to rely on an (undocumented) implementation 
detail like that. I'm pretty sure I'm not doing that in any code where I 
introduced use of the class.

Since I removed the DoublyLinkedListLink constructor about a month ago and 
haven't encountered any new unexplained crashes since, I'm somewhat 
optimistic. I guess it doesn't harm to grep a bit through the sources, though.

CU, Ingo

Other related posts: