diff options
| author | Joseph Myers <joseph@codesourcery.com> | 2014-12-10 00:41:19 +0000 |
|---|---|---|
| committer | Joseph Myers <joseph@codesourcery.com> | 2014-12-10 00:41:19 +0000 |
| commit | 2084e7ca4d344c39eb39e53848b51b5d84444414 (patch) | |
| tree | 2c2c2be22115d5b467c637944a34a410bc1714ac /ChangeLog | |
| parent | df7ecc6bd2a3b93441287eff5ad9feed84df7373 (diff) | |
| download | glibc-2084e7ca4d344c39eb39e53848b51b5d84444414.tar.gz glibc-2084e7ca4d344c39eb39e53848b51b5d84444414.tar.xz glibc-2084e7ca4d344c39eb39e53848b51b5d84444414.zip | |
Add macros for diagnostic control, use for scanf %a tests.
In <https://sourceware.org/ml/libc-alpha/2014-11/msg00326.html>, Roland requested internal macros for use of "#pragma GCC diagnostic". This patch adds such macros and uses them to disable -Wformat warnings for some code testing GNU scanf %as where GCC expects C99 scanf %a (several other stdio tests currently use -Wno-format to disable warnings). Limitations in GCC's diagnostic pragmas require separate macros before and after the code generating the warnings, rather than a single macro taking that code as an argument. The macros are named DIAG_*_NEEDS_COMMENT to emphasise to reviewers the need for a comment accompanying any use of them (such comments may however just appear once for several uses of the macros for the same issue in the same file). I put a GCC version in the arguments to DIAG_IGNORE_NEEDS_COMMENT, as that seems something useful to grep for when obsoleting support for an old GCC version and needing to decide if warning-disabling code is still relevant. These macros should be usable for replacing existing -Wno-* use in makefiles (as also suggested by Roland), though I have no plans to work on that (only on use of the macros in cases where warnings are currently present that need disabling to use -Werror). Tested for x86_64. * include/libc-internal.h (DIAG_PUSH_NEEDS_COMMENT): New macro. (DIAG_POP_NEEDS_COMMENT): Likewise. (_DIAG_STR1): Likewise. (_DIAG_STR): Likewise. (DIAG_IGNORE_NEEDS_COMMENT): Likewise. * stdio-common/bug21.c: Include <libc-internal.h>. (do_test): Disable -Wformat around call to sscanf. * stdio-common/scanf14.c: Include <libc-internal.h>. (main): Disable -Wformat around some calls to scanf functions.
Diffstat (limited to 'ChangeLog')
| -rw-r--r-- | ChangeLog | 12 |
1 files changed, 12 insertions, 0 deletions
diff --git a/ChangeLog b/ChangeLog index 7cd1a45ea3..cfbaf86faa 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,15 @@ +2014-12-10 Joseph Myers <joseph@codesourcery.com> + + * include/libc-internal.h (DIAG_PUSH_NEEDS_COMMENT): New macro. + (DIAG_POP_NEEDS_COMMENT): Likewise. + (_DIAG_STR1): Likewise. + (_DIAG_STR): Likewise. + (DIAG_IGNORE_NEEDS_COMMENT): Likewise. + * stdio-common/bug21.c: Include <libc-internal.h>. + (do_test): Disable -Wformat around call to sscanf. + * stdio-common/scanf14.c: Include <libc-internal.h>. + (main): Disable -Wformat around some calls to scanf functions. + 2014-12-09 Torvald Riegel <triegel@redhat.com> * sysdeps/unix/sysv/linux/sh/pthread_once.S: Remove file. |