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>