diff options
author | Tony Cook <tony@develop-help.com> | 2021-08-03 16:39:45 +1000 |
---|---|---|
committer | Karl Williamson <khw@cpan.org> | 2021-08-03 07:54:15 -0700 |
commit | 5f3563911b9fd00ff487932f3476fc6b56d6a002 (patch) | |
tree | 607343c362d9db838b2a3ccb8b8f7b55ccfcde03 /scope.h | |
parent | 09927302e9757ebc2c12374c7680b7bf61fde2ce (diff) | |
download | perl-5f3563911b9fd00ff487932f3476fc6b56d6a002.tar.gz |
use CLANG_DIAG_IGNORE_STMT instead of GCC_DIAG_IGNORE_STMT
It turns out gcc wasn't warning on this code, but older gcc (as
included in debian buster) *does* warn on the switch which it
doesn't recognise.
Newer gcc does recognise the -Wstring-compare switch, but it controls
warning on a different construct, so there's no reason to present it
to gcc.
Diffstat (limited to 'scope.h')
-rw-r--r-- | scope.h | 4 |
1 files changed, 2 insertions, 2 deletions
@@ -208,11 +208,11 @@ scope has the given name. C<name> must be a literal string. STMT_START { \ DEBUG_SCOPE("LEAVE \"" name "\"") \ if (PL_scopestack_name) { \ - GCC_DIAG_IGNORE_STMT(-Wstring-compare); \ + CLANG_DIAG_IGNORE_STMT(-Wstring-compare); \ assert(((char*)PL_scopestack_name[PL_scopestack_ix-1] \ == (char*)name) \ || strEQ(PL_scopestack_name[PL_scopestack_ix-1], name)); \ - GCC_DIAG_RESTORE_STMT; \ + CLANG_DIAG_RESTORE_STMT; \ } \ pop_scope(); \ } STMT_END |