Date: Wed, 04 Jun 2003 20:27:25 -0700 From: Peter Wemm <peter@wemm.org> To: "Daniel C. Sobral" <dcs@tcoip.com.br> Cc: Mike Barcroft <mike@FreeBSD.org> Subject: Re: cvs commit: src/tools/regression/usr.bin/sed regress.s3.out regress.s4.out regress.s5.out regress.sg.out regress.sh Message-ID: <20030605032725.336B42A8C1@canning.wemm.org> In-Reply-To: <3EDE3905.7020506@tcoip.com.br>
next in thread | previous in thread | raw e-mail | index | archive | help
"Daniel C. Sobral" wrote: > Mike Barcroft wrote: > > Tony Finch <fanf@FreeBSD.org> writes: > > > >>fanf 2003/06/04 08:26:24 PDT > >> > >> FreeBSD src repository > >> > >> Modified files: > >> tools/regression/usr.bin/sed regress.sh > >> Added files: > >> tools/regression/usr.bin/sed regress.s3.out regress.s4.out > >> regress.s5.out regress.sg.out > >> Log: > >> Add tests for the behaviour of substitution when the regex can match > >> a zero-length string. This shows bugs in the s///g and s///2+ cases. > > Let me add my thanks too. This kind of bug is the worst kind possible. > Particularly when you _don't_ detect it, and end up with flawed data. In this case however, the cure is worse than the disease. World wont build anymore since sed is now broken. 'make world' seems to be a better basic regression test in this instance. Cheers, -Peter -- Peter Wemm - peter@wemm.org; peter@FreeBSD.org; peter@yahoo-inc.com "All of this is for nothing if we don't go to the stars" - JMS/B5
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20030605032725.336B42A8C1>