Branch: refs/heads/yves/fix_diag_t Home: https://github.com/Perl/perl5 Commit: 7dd3f8f9ad46a536daaaa74fc03ad8d5a0e83517 https://github.com/Perl/perl5/commit/7dd3f8f9ad46a536daaaa74fc03ad8d5a0e83517 Author: Yves Orton <demer...@gmail.com> Date: 2022-09-05 (Mon, 05 Sep 2022)
Changed paths: M dquote.c M op.c M os2/os2ish.h M perl.h M t/porting/diag.t Log Message: ----------- porting/diag.t - improved parsing a bit The "multiline" logic of diag.t was getting confused by define statements that would define a symbol to call an error function but not end in ";", this would then slurp potentially many lines errorenously, potentially absorbing more than one message. The multi-line logic also would undef $listed_as and lose the diag_listed_as data in some circumstances. Fixing those issues revealed some interesting cases. To fix one of them I defined a new noop macro in perl.h to help: PERL_DIAG_WARN_SYNTAX(), which helps the diag.t parser identify messages without needing to be actually part of a specific message line. These macros are noops, they just return their argument, but they help hint to diag.t what is going on. Maybe in the future this can be reworked to be more generic, there are other similar cases that are not covered. Interestingly fixing this bug meant that at least one message that used to be erroneously picked up was no longer identified or tested. This was replaced with a PERL_DIAG_DIE_SYNTAX() wrapper.