From owner-cvs-all Thu Sep 6 2:32:41 2001 Delivered-To: cvs-all@freebsd.org Received: from columbus.cris.net (columbus.cris.net [212.110.128.65]) by hub.freebsd.org (Postfix) with ESMTP id B9F9A37B406; Thu, 6 Sep 2001 02:31:02 -0700 (PDT) Received: from ark.cris.net (ark.cris.net [212.110.128.68]) by columbus.cris.net (8.9.3/8.9.3) with ESMTP id MAA00109; Thu, 6 Sep 2001 12:30:47 +0300 (EEST) Received: (from phantom@localhost) by ark.cris.net (8.11.1/8.11.1) id f869UJs25770; Thu, 6 Sep 2001 12:30:19 +0300 (EEST) Date: Thu, 6 Sep 2001 12:30:19 +0300 From: Alexey Zelkin To: Mark Peek Cc: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/lib/libc/stdlib strfmon.c Makefile.inc Message-ID: <20010906123019.A23173@ark.cris.net> References: <200109051850.f85Io3o99434@freefall.freebsd.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii X-Mailer: Mutt 1.0i In-Reply-To: ; from mark@whistle.com on Wed, Sep 05, 2001 at 07:07:09PM -0700 X-Operating-System: FreeBSD 3.5-STABLE i386 Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG hi, I am sorry. I've fixed this problem in local tree during my buildworld testing, but forgot to update development version of strfmon(3) :( Thanks for notice, fixed. On Wed, Sep 05, 2001 at 07:07:09PM -0700, Mark Peek wrote: > At 11:50 AM -0700 9/5/01, Alexey Zelkin wrote: > >phantom 2001/09/05 11:50:02 PDT > > > > Modified files: > > lib/libc/stdlib Makefile.inc > > Added files: > > lib/libc/stdlib strfmon.c > > Log: > > Add strfmon(3) implementation. It still contains few XXX's because I lost > > my last version of this work due to HDD crash, but this version cleanly > > passed all POSIX and SuSv2 tests. I am working on testing scripts which > > should test this implementation against all locales and surely more fixes > > will come soon. > > > > Reviewed by: ache, silence at -audit & -developers > > > > Revision Changes Path > > 1.29 +4 -4 src/lib/libc/stdlib/Makefile.inc > > > This broke my -current buildworld: > > cc -nostdinc -O -pipe -DLIBC_RCS -DSYSLIBC_RCS -I/usr/src/lib/libc/include -I/usr/src/lib/libc/../../include -DLIBC_MAJOR=5 -D__DBINTERFACE_PRIVATE -DINET6 -I/usr/obj/usr/src/lib/libc -DPOSIX_MISTAKE -I/usr/src/lib/libc/../libc/locale -DBROKEN_DES -DPORTMAP -DDES_BUILTIN -DYP -DHESIOD -I/usr/obj/usr/src/i386/usr/include -c /usr/src/lib/libc/../libc/stdlib/strfmon.c -o strfmon.o > /usr/src/lib/libc/../libc/stdlib/strfmon.c: In function `strfmon': > /usr/src/lib/libc/../libc/stdlib/strfmon.c:302: syntax error before `else' > /usr/src/lib/libc/../libc/stdlib/strfmon.c:233: label `end_error' used but not defined > /usr/src/lib/libc/../libc/stdlib/strfmon.c:162: label `format_error' used but not defined > /usr/src/lib/libc/../libc/stdlib/strfmon.c:144: label `e2big_error' used but not defined > > Below is a patch that matches the code put up for review at > http://phantom.cris.net/~phantom/strfmon04.tgz > > > Mark > > > > Index: lib/libc/stdlib/strfmon.c > =================================================================== > RCS file: /cvs/freebsd/src/lib/libc/stdlib/strfmon.c,v > retrieving revision 1.1 > diff -u -u -r1.1 strfmon.c > --- lib/libc/stdlib/strfmon.c 2001/09/05 18:50:02 1.1 > +++ lib/libc/stdlib/strfmon.c 2001/09/06 01:50:12 > @@ -297,10 +297,11 @@ > PRINT(' '); > > if (sign_posn == 0) { > - if (flags & IS_NEGATIVE) > + if (flags & IS_NEGATIVE) { > PRINT('('); > - else > + } else { > PRINT(' '); > + } > } > > if (cs_precedes == 1) { > To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message