Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 6 Sep 2001 12:30:19 +0300
From:      Alexey Zelkin <phantom@FreeBSD.org>
To:        Mark Peek <mark@whistle.com>
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>
In-Reply-To: <p05100301b7bc89dbcf10@[207.76.207.129]>; from mark@whistle.com on Wed, Sep 05, 2001 at 07:07:09PM -0700
References:  <200109051850.f85Io3o99434@freefall.freebsd.org> <p05100301b7bc89dbcf10@[207.76.207.129]>

next in thread | previous in thread | raw e-mail | index | archive | help
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




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20010906123019.A23173>