Date: Sat, 21 Sep 2002 08:32:16 -0700 From: Eric Melville <eric@FreeBSD.org> To: Mike Barcroft <mike@FreeBSD.org> Cc: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/usr.bin/sed process.c Message-ID: <20020921083216.A81079@FreeBSD.org> In-Reply-To: <20020920235606.B51443@espresso.q9media.com>; from mike@FreeBSD.org on Fri, Sep 20, 2002 at 11:56:06PM -0400 References: <200209201940.g8KJeNqj059268@freefall.freebsd.org> <20020920194544.A51443@espresso.q9media.com> <20020920182706.A32662@FreeBSD.org> <20020920235606.B51443@espresso.q9media.com>
next in thread | previous in thread | raw e-mail | index | archive | help
> > > Sounds like a compiler bug. This probably isn't correct for FreeBSD, > > > since the system compiler doesn't have this bug. > > > > Try -Wall -Werror. > > The bug doesn't show up unless -Wuninitialized is specified. I > thought I tried this before e-mailing you. In any event, the compiler > is correct, p could have been uninitialized at line 198 (of the latest > rev) in the psl == 0 case. Well, yes, and -Wuninitialized is turned on by the -Wall option. What are you trying to say? At first it's a compiler bug, now the compiler is correct? If there is some way you'd prefer this code to look, it would be better to be more forward about it. I really don't mind how it looks as long as it can be compiled when both -Wall and -Werror are specified. 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?20020921083216.A81079>