[openbox] [PATCH] new timers for ob3

Ben Jansens ben at orodu.net
Tue Jan 14 03:50:06 EST 2003


ManMower <manmower at is.fuckingabitch.com> writes:

> The attached patch replaces the current timer implementation in ob3
> 
> these timers are somewhat simpler.  there is only one (static) timer
> queue, and timers have no explicit start/stop controls.
> 
> a = new otk::Timer(callback, timeout, data);
> starts a timer that is immediately scheduled in its constructor and fires
> every timout milliseconds.
> 
> delete a;
> stops a running timer.
> 
> This code may be used under any license the openbox maintainer desires.

Thanks a lot, G! One step closer to shedding the blackbox skin.

Questions re your patch:
a) Why the added unistd.h include? What function is it for?

b) Why the constructor order of (callback, timeout, callbackdata)? I
think (timeout, callback, callbackdata) makes more sense.

Fabulous work :)

Ben
-- 
I am damn unsatisfied to be killed in this way.

http://www.icculus.org/openbox/




More information about the openbox mailing list