Re: slimp3.pl plugins patches and comments

I made a reply similar to the following from work, but I guess the outgoing 
mail queue there is backed up.

My basic point is that functions should be named for what they do, not for 
compliance with "well-known function names", which is just a pretty way of 
saying the button names.

For example, in Shooter, there is a function which fires -- named 'play';
in SlimTris there is a function which rotates -- broken into two functions 
named 'play' and 'add';
in Rescan there is a function which starts rescanning -- named 'play'

In my opinion, the function in Shooter should be named 'fire', the 
functions in SlimTris should be reduced to a single 'rotate' function, 
which takes a parameter to determine direction, and the function in Rescan 
should be named 'rescan'.

Using "well-known function names" adds a third layer of mapping in the 
developers mind between the functions name and what it actually does.

There is also the possibility that the plugin developer would want to have 
a button behave differently to various pressing styles which may not be 
mapped in the default [common] section.

Robert

At 02:46 PM 2/14/2003 -0800, you wrote:
> >
>I am still not following.
>
>The default+custom mapping files map a physical button name (which
>corresponds to a IR code for a given remote)
>to a function name.  We have default mappings under COMMON for each of
>the buttons on our remotes to well-known function names.
>
>Plugin authors write their plugins and create functions based on these
>well-known names.
>
>Under what circumstances would a plugin author need to alter the map
>away from the standard function names?
>
>-dean


------------------------ Yahoo! Groups Sponsor ---------------------~-->
Get 128 Bit SSL Encryption!
http://us.click.yahoo.com/FpY02D/vN2EAA/xGHJAA/rIp0lB/TM
---------------------------------------------------------------------~->

To unsubscribe from this group, send an email to:
[email protected]

 

Your use of Yahoo! Groups is subject to http://docs.yahoo.com/info/terms/