duplicated/bad output from apm(8)
Maxime Henrion
mux at freebsd.org
Wed Jun 30 12:56:52 PDT 2004
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
> @@ -226,7 +226,6 @@
> 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
More information about the freebsd-current
mailing list