Klaus Schmidinger wrote:
I would suggest to rename NextWakeupEvent() to WakeupTime(), because
there isn't really an "event" involved, and the word "event is generally
used in a different context (for EPG events). Plus, the individual
plugin only has a single wake-up time, so there's no need for "Next".
Accordingly GetNextWakeupEventPlugin() should be renamed to
GetNextWakeupPlugin(). Here the "Next" makes sense, because the
function returns the plugin with the earliest wake-up time. The
"Time" part could be dropped because the function returns a cPlugin*
and there is no time involved.
I agree, s/event/time/g makes things a lot more consistent. I didn't
like GetNextWakeupEventPlugin either.
I also renamed the setup parameter NextWakeupEvent to NextWakeupTime,
and renamed some local variables accordingly. I can provide a patch if
needed.
Cheers,
Udo
_______________________________________________
vdr mailing list
vdr@linuxtv.org
http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr
