[Pkg-xfce-devel] Bug#629265: Bug#629265: Bug#629265: Bug#629265: Broken empty battery icon on system tray

[Kim Holviala]

On 2011-06-05 14:43, Yves-Alexis Perez wrote:

>> ** (xfce4-power-manager:11351): WARNING **: Unable to load icon :
>> \u0018a\u001e\xad\xdb\u007f : Icon '\u0018a\u001e\xad\xdb\u007f' not
>> present in theme
>>
>> Anyway, that looks like a pointer problem to me - if you decode the
>> UTF-8 string it makes no sense.
>
> Yeah definitely. But nothing around that? Because the error is from
> xfpm_icon_load() which is called quite a few time so... You might want
> to instrument that function to see exactly where does the icon pointer
> comes from but...

Nothing around that log entry which would have helped...

I traced the code a bit and found function xfpm_battery_refresh_icon() 
which looks a bit fishy. It's the function that converts battery 
percentage to an icon name. I added some additional debug outputs to it 
to see what it really does - but I think the problem is an if-elseif 
without the final else which then proceeds to go all the way through. So 
an simple final else for the "empty battery" case will most likely fix 
this bug.

Unfortunately I forgot my laptop plugged in, so now I have to wait a 
couple of hours for that first battery to be fully depleted....



- Kim