Re: duplicated/bad output from apm(8)

From: Maxime Henrion <mux_at_freebsd.org>
Date: Wed, 30 Jun 2004 21:56:49 +0200
Niki Denev wrote:
> It seems that one of the last few commits 
> to 'usr.sbin/apm/apm.c' introduced 
> an extra printf for the battery status output.
> And the output is like this :
> 
>  Number of batteries: 1
>  Battery 0:
>          Battery status:         Battery Status: charging
>  ^^^^^^^^^^^^^^^^^^^^^^^
>  This is extra
> 
> here is the patch that i used to fix
> the output for me and my precious X31 :)
> 
> ---------------------------------------------------------------------------- 
> --- usr.sbin/apm/apm.c.orig	Wed Jun 30 22:26:28 2004
> +++ usr.sbin/apm/apm.c	Wed Jun 30 22:35:12 2004
> _at__at_ -226,7 +226,6 _at__at_
> 				if (ioctl(fd, APMIO_GETPWSTATUS, &aps) == -1)
> 					continue;
> 				printf("Battery %d:\n", i);
> -				printf("\tBattery status: ");
> 				if (aps.ap_batt_flag <= 255 &&
> 				    (aps.ap_batt_flag & 
> 				    APM_BATT_NOT_PRESENT)) {
> 					printf("not present\n");

Indeed.  I just committed your patch, thanks!

Cheers,
Maxime
Received on Wed Jun 30 2004 - 17:56:52 UTC

This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:37:59 UTC