On Thu, 2005-11-24 at 22:56 +0100, Danny Kukawka wrote: Looks good, other than a totally cosmetic line issue: > chargeRate = battery_info->last_chargeRate; > } > else { > chargeRate = ((chargeLevel - ... I've tested this patch for a couple of days as well, with no ill effects. Richard.