[interfacekit] Re: Some news

> Some good and bad news:
> 
> The good: I was able to run all the BApplication tests using our 
> app_server and
> registrar and our libbe.so.
Nice.

> I needed to fix some stuff in ServerApp, though, and something in 
> CursorManager.cpp.
> 
> I won't commit the patches as I'm not sure about the code in there, 
> and it would
> be better than you (DW or Adi) do the work.
> 
> In ServerApp.cpp, ServerApp::~ServerApp(). I had to comment the 
> following lines:
> 
> if (fAppCursor)
>   delete fAppcursor.
> 
> This lead to a crash, because, later, 
> CursorManager::RemoveAppCursor() is
> called, which deletes that cursor again.
> 
> The other stuff which is broken is the above function. You need to 
> iterate the
> list in reverse order. Otherwise, removing items as you do will lead 
> to  out of
> bounds writes.
> 
> I can't really understand well that code, in fact I don't understand 
> why there
> is need for RemoveAppCursor.
Off the top of my head, I can't either, but I know I wouldn't have put 
it in there without a halfway decent reason to do so. I wrote the 
offending code, so I'll see if I can't get around to fixing it in the 
next day or two. 

> Anyway, the bad news is that I thought that the BMessage issue was 
> caused by the
> use of my BDataIO related classes, but trying to remove them from the 
> build
> didn't help.
Bummer. I know the feeling. Thankfully, though, I know that it'll work 
out eventually. :)

--DW

Other related posts: