Date: Wed, 30 Jun 2004 21:56:49 +0200 From: Maxime Henrion <mux@freebsd.org> To: Niki Denev <nike_d@cytexbg.com> Cc: current@freebsd.org Subject: Re: duplicated/bad output from apm(8) Message-ID: <20040630195649.GB9228@elvis.mu.org> In-Reply-To: <cone.1088624984.490789.732.1001@phobos.totalterror.net> References: <cone.1088624984.490789.732.1001@phobos.totalterror.net>
next in thread | previous in thread | raw e-mail | index | archive | help
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
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20040630195649.GB9228>