hijack feature 2

another idea.... i just got ben's fan controller installed and running.... but it could be nice to have a menu option to set the vaules for turning the fan on and off... maybe also bee able to tell what temp ben's controller is reading... along with a value that correct that reading.. just like in high temp warning

Menu thingies take up precious RAM. Anything in the kernel gets permanently loaded in RAM, reducing the amount for the unit's primary function of playing tunes.

So, when it comes to new features, I mostly don't put them into the default build unless there's widespread interest, or unless somebody the community really reveres has a nagging need for it (eg. Hugo, Patrick, Tony ..). Or unless *I* want it on *my* players.

The fan controllers are quite rare, so they don't pass that test, yet.

Cheers

and doing it like this:

[hijack]
;@MENUEXEC Set_Date_Time /rdate -s 10.0.0.2 &>/dev/null

does not take it out of RAM? but i guees you dont change vaules that much... just sounded like a good idea

The line above is tiny, and is not permanently in RAM. The program it calls (rdate), is not tiny, and is on disk, not permanently in RAM either.

Cheers

Basically, the golden rule for kernel code is, don't put it into the kernel unless it is extraordinarily difficult to do from userspace and simple to do in kernel space.

With Hijack, I extend that rule, to include things that are of wide (or simply my own personal) interest, which are not trivial to do in userspace.

-ml

Aw shucks...