Bug#669020: fdpowermon doesn't handle battery state 'Full'

[gregor herrmann]

Package: fdpowermon
Version: 1.5
Severity: minor
Tags: patch

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA256

Today fdpowermon didn't show me an icon but only an empty space.
After a little debugging I found that it doesn't handle the current
output of acpi -b:

% acpi -b
Battery 0: Full, 100%

Find attached a quick patch; I hope I understand the logic more or
less right :)


Cheers,
gregor

- -- System Information:
Debian Release: wheezy/sid
  APT prefers unstable
  APT policy: (990, 'unstable'), (500, 'experimental'), (500, 'testing'), (500, 
'stable')
Architecture: amd64 (x86_64)

Kernel: Linux 3.3.0-trunk-amd64 (SMP w/4 CPU cores)
Locale: LANG=C, LC_CTYPE=de_AT.utf8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/bash

Versions of packages fdpowermon depends on:
ii  acpi              1.6-1
ii  fdpowermon-icons  1.5
ii  libgtk2-perl      2:1.242-1
ii  perl              5.14.2-9

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.12 (GNU/Linux)

iQIcBAEBCAAGBQJPjDpCAAoJELs6aAGGSaoGCZEP/iWJt7TiFEGNgmmxSDs0LZAH
DZbKlSGbLQWZpPA+3OS+5P1zPW7SbKFfB3bKRLQfv1ciWgUNU2YMDiIttswg22Cd
QzgInAK1gMAGHw1aQBBCnryBXBoqBA0qUFw0tyZe9hevoF6pq7hwH3/uTvTssbSk
N0fkoXgpbbKS1ZVUzuk11mKIdhao58nD3ZpQf0QHwFD0UalNgWmhW65jybU5X7e4
02P7eGDYZ6LFSZcT3JtZT2V4W7Q6TO/WyyoD+9Akkmh8fp4qQ2pl/7o6b/G2RqPB
WYrBbIOftOHOlARQmDXaBlSQNRJ7c6TWxV3WoGXv0hQ6iRU555HrIrKe6q/Hm+LZ
+wziC2xkevY3MxuDFN0CK70XXRk/pd2YQdDKIsS0Ve0nKsz8NZol20yLEJVyfM+X
roYzarqCRmXiCeTmR/hPYvPuoh5IRYFKt7guLRRImu9xavAntb/709WKH6ly4MpY
ycWVt/eNEoFn+1ForVfEA94d00P5wnRBKI1Ppdn15DOq+l/QV77/tsu7CGZWMrMx
JYMi1LAhRaw02xozBnDTVfJoHxoqUfUO/oHZj6Al1csvX0XMJOY4WBdftScREQy7
j0I/7QV+5CNxaaLiPs4MtyGMusyaE17E/FoVkSp2/UYxz5GfmBZjlnVzsstxPeGC
LXp+NIoma+OvVEpvwLmX
=X263
-----END PGP SIGNATURE-----

--- fdpowermon.orig	2012-04-16 16:47:45.415892396 +0200
+++ fdpowermon	2012-04-16 17:22:27.410760405 +0200
@@ -25,7 +25,7 @@
 	open my $acpi, "acpi -b |";
 	while ($acpi_output = <$acpi>) {
 		chomp $acpi_output;
-		if ($acpi_output =~ /^Battery (\d): ((Dis)?[Cc]harging|Unknown), ((\d)+)%(, ([\d:]*))?/) {
+		if ($acpi_output =~ /^Battery (\d): ((Dis)?[Cc]harging|Unknown|Full), ((\d)+)%(, ([\d:]*))?/) {
 			$bat = $1;
 			$state = $2;
 			$level = $4;
@@ -44,6 +44,9 @@
 	} elsif($state =~/Charging/) {
 		$step = $theme->{charging};
 		$charging = 1;
+	} elsif($state =~/Full/) {
+		$step = $theme->{discharging};
+		$charging = 1;
 	} else {
 		$step = $theme->{charging};
 	}