Rockbox.org home
release
dev builds
extras
themes manual
wiki
device status forums
mailing lists
IRC bugs
patches
dev guide



Rockbox mail archive

Subject: Re: 1 standard generic callback system

Re: 1 standard generic callback system

From: Linus Nielsen Feltzing <linus_at_haxx.se>
Date: Tue, 05 Jun 2007 18:11:21 +0200

Jonathan Gordon wrote:
> Questions, comments?

I think it sounds like a good idea. The implementation looks clean and
simple.

1) Why this:

+#define register_ata_idle_func(f) \
+ register_event_callback(EID_ATA_DISK_SPINUP, (event_callback)f)

Why not replace the calls in the ATA code?

2) Question:

> + if (run)
> + callbacks[i].function(NULL);

Is it always true that the callback wants a NULL argument in this case?

Linus
Received on 2007-06-05


Page was last modified "Jan 10 2012" The Rockbox Crew
aaa