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

Reply via email to