From: Tobias Heimann <tobias.heimann_at_gmx.net>
Date: Fri, 05 Jan 2007 01:27:41 +0100

Linus Nielsen Feltzing wrote:
> Tobias Heimann wrote:
>> IMHO it would be more logical to spread the highest interval as the
>> other ones, e.g. using:
>>
>> int l = MIN(battery_level(), 99);
>> ...
>> *intval = limit * l / 100 + 1 + 1;
>>
>> The volume display interval has the same issue btw.
>
> Sounds OK to me. Care to submit a patch?
>
> Linus
>

Done. It's under http://www.rockbox.org/tracker/task/6506


Kaggen wrote:
> A reason for this behavior might be that when charging it is probably
> nice actually have a fully charged battery when it displays "full"?
> Instead of unplugging it when it just reached 90% or whatever in
> belief that it's full.

Mmm, maybe. I only know the H120 which has a LED to inform about the
charging state... In case the above patch is not included in CVS, this
special battery icon behaviour should be explained in the manual.

Tobias
Received on 2007-01-05