Front page | perl.perl5.porters |
Postings from October 2013
On Wed Oct 23 03:35:04 2013, davem wrote: > > > > The answer is plain as day in > > > > http://www.nntp.perl.org/group/perl.daily- > build.reports/2013/10/msg152778.html > > > > I won't provide a patch to protest discrimination against C89 :p > > It may be plain to you, but not to me. Care to elaborate? > A lack of knowledge of C89, since AFAIK, most P5Pers use GCC in C++ mode here, and if my C++ knowledge is correct, they will never see the bug. Commit http://perl5.git.perl.org/perl.git/commit/db17973a7879fcb6ada1024d1c72e99a944746d1 is responsible. I'll leave it to khw to realize his mistake in that commit and fix all the warnings in regcomp.c in http://www.nntp.perl.org/group/perl.daily-build.reports/2013/10/msg152778.html . I am upset because the only reason I saw this is because I looked at my screen because my TV show went to commercials. The smoke report still "passes" even though its outputting severe warnings. The timing of commercials on the TV is not a reliable way to check for bugs in a make test. IMO if it wasn't for a TV commercial this would have silently gone into 5.20. Anyone agree or disagree? Maybe those SV leak warnings should be fatal on DEBUGGING builds, not warnings? -- bulk88 ~ bulk88 at hotmail.com --- via perlbug: queue: perl5 status: open https://rt.perl.org:443/rt3/Ticket/Display.html?id=120314Thread Previous | Thread Next