[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