[haiku-doc] Re: Finished initial docs for BHandler

  • From: "Niels Reedijk" <niels.reedijk@xxxxxxxxx>
  • To: haiku-doc@xxxxxxxxxxxxx
  • Date: Sat, 5 Jan 2008 17:35:11 +0100

Hi Axel,

2008/1/3, Axel Dörfler <axeld@xxxxxxxxxxxxxxxx>:
> "Niels Reedijk" <niels.reedijk@xxxxxxxxx> wrote:
> > As a post-christmas gift, here
> > (http://factory.haiku-os.org/documentation/Haiku_Book_doxygen/html/classBHandler.html)
> > > is the initial documentation for BHandler.
>
> Nice, and thanks!
>
> I just have two suggestions:
> 1) the warning to not clobber the message in MessageReceived() could
> have a hint that you can detach the current message using your looper's
> DetachCurrentMessage().

Thanks, I will add that suggestion.

> 2) the description to StartWatching[All](BMessenger ...) is a bit
> misleading: you actually subscribe yourself with the "target" to watch
> the specified state on the target handler, not vice versa. Same for
> StopWatching[All](BMessenger ...).

Ah, that makes a lot more sense with the code. So if Handler B wants
to watch Handler A, Handler B does this->StartWatching(A)?

Thanks for looking at the docs, I will update them later today.

Niels

Other related posts: