[patch] Reworking HAL's battery handling, was:Trivial error message fix

[David Zeuthen]

On Mon, 2006-11-27 at 16:38 +0000, Richard Hughes wrote:
> On Mon, 2006-11-06 at 18:18 -0500, Stu Hood wrote:
> > Quick nitpicks:
> >       * @330 - Slight indentation problem
> >       * @364 - TRUE in boolean statement
> >       * @400 - Check against some ratio of bat->last_full instead: For
> >         instance, (rate > 0.25 * last_full)? (huge/server batteries
> >         could have rate >50000)
> >       * @923 - Broken comment
> > Otherwise, brilliant!
> 
> David, what do you think? I've fixed these little nits, and really want
> to get this in well before the next HAL release.

Sure, I thought we were waiting for Danny to OK this? You and Danny are
probably better at reviewing this than me... 

Also, looking at the patch again.. since the new code can remove
properties.. is this documented in the spec? Do we do this already? Does
g-p-m and kpowersave cope with that? 

     David