[Haiku-commits] r29488 - in haiku/trunk/src: add-ons/input_server/filters/screen_saver bin/screen_blanker

[Ryan Leavengood]

On Fri, Mar 13, 2009 at 4:40 AM, Axel Dörfler <axeld at pinc-software.de> wrote:
>
> Am I right that fEnabled is never set to false now? Ie. the screen
> saver can only run one now?

The ScreenSaverController class watches app launching and quitting and
will set fEnabled to false when the screen_blanker quits. It works
believe me :)

> If you're using the pointer like this, you can as well write:
>        fWindow->SendMessage(&enable);

OK I wasn't sure.

> You ignore mouse events because you're the top view anyway,  I guess?

That was something Rene suggested, so I guess so.

> A new window gets a mouse moved event if the cursor is over it (so that
> it can do whatever it does in this case). Since you can assume this
> always to be true for a full screen window, why not just ignore that
> first one?

For some reason I seemed to get two such events, though when I got
some help from Rene he was only seeing one I think. I was testing in
VMware. But when I added a counter to ignore the first few events then
keyboard input was ignored for the first event. I can play with it
some more.

> Since those are specially made in the app_server, we could even add an
> extra flag to them, though.

That might help too.

Regards,
Ryan