retitle 726525 should parse /sys directly, rather than calling acpi severity 726525 wishlist thanks
Op 16-10-13 14:40, Paul Slootman schreef: > Package: fdpowermon > Version: 1.5 > Severity: normal > > When my battery is full, fdpowermon still shows "Charging, 98%". This was fixed in fdpowermon 1.6 (which did not make stable only because it missed the freeze due to my procrastination) [...] > Perhaps it would also be nicer to directly access the > /sys/class/power_supply/* stuff? That would prevent spawning > a couple of processes every 3 seconds. Yeah, I should probably do that. Polling really isn't a good use of resources, even if we do it only once every three seconds. The reason I didn't was lazyness, but that's really not a good enough excuse. I'll look into this when next I have some time. For now, retitling and changing severity so this part of the bug stands out ;-) > Fall back to acpi -b > if the system has an unexpected /sys/class/power_supply/ layout. Since acpi itself does little more than reading files in that directory, I doubt I'll have more luck in that case (unless there's a bug in my code, of course, which should then be fixed). Since adding fallback code would add complexity to fdpowermon _and_ result in code paths which would see very little testing, I'll just drop that part instead. -- This end should point toward the ground if you want to go to space. If it starts pointing toward space you are having a bad problem and you will not go to space today. -- http://xkcd.com/1133/ -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org