[patch] Patch documentation with the new lcdpanel capability

David Zeuthen david at fubar.dk
Wed Aug 31 08:07:28 PDT 2005


On Tue, 2005-08-30 at 17:45 +0100, Richard Hughes wrote:
> Good to commit?
> 

It's not a good idea to use properties...

> 
> +              <entry><literal>lcdpanel.percent</literal> (int)</entry>
> +              <entry><literal>lcdpanel.current_level</literal> (int)</entry>

As discussed on IRC yesterday this really need to be method calls, e.g.
GetLCDBrightness()... Because the user may use hotkeys to change
brightness and we don't want to poll

Btw, I think Timo made that point too some weeks ago on the mailing
list, see this 

 http://lists.freedesktop.org/archives/hal/2005-August/002984.html

and surrounding messages.

    David




More information about the hal mailing list