diff options
| author | Philipp Stephani <phst@google.com> | 2017-06-13 13:55:44 +0200 |
|---|---|---|
| committer | Philipp Stephani <phst@google.com> | 2017-06-13 13:56:46 +0200 |
| commit | e408e9aa030a00cb1a61acdc729e8d6786b25fe3 (patch) | |
| tree | 2173ff44df3fd9d906721137eada524c9e029c0d /lib | |
| parent | cc8aa484cdab6b2f33a8c95a5778193c762412b9 (diff) | |
| download | emacs-e408e9aa030a00cb1a61acdc729e8d6786b25fe3.tar.gz emacs-e408e9aa030a00cb1a61acdc729e8d6786b25fe3.tar.bz2 emacs-e408e9aa030a00cb1a61acdc729e8d6786b25fe3.zip | |
Silence two Clang warnings by introducing additional local variables
* lib/strftime.c (libc_hidden_def):
* lib-src/make-docfile.c (put_filename): Introduce local variables to
silence Clang warnings.
Diffstat (limited to 'lib')
| -rw-r--r-- | lib/strftime.c | 27 |
1 files changed, 16 insertions, 11 deletions
diff --git a/lib/strftime.c b/lib/strftime.c index 99bee4ef978..18c899d2117 100644 --- a/lib/strftime.c +++ b/lib/strftime.c @@ -1123,18 +1123,23 @@ __strftime_internal (STREAM_OR_CHAR_T *s, STRFTIME_ARG (size_t maxsize) if (modifier == L_('E')) goto bad_format; - number_value = ns; - if (width == -1) - width = 9; - else - { - /* Take an explicit width less than 9 as a precision. */ - int j; - for (j = width; j < 9; j++) - number_value /= 10; - } + { + /* Use a new variable here instead of reusing number_value + because Clang complains about the self-assignment + generated by DO_NUMBER. */ + ptrdiff_t n = ns; + if (width == -1) + width = 9; + else + { + /* Take an explicit width less than 9 as a precision. */ + int j; + for (j = width; j < 9; j++) + n /= 10; + } - DO_NUMBER (width, number_value); + DO_NUMBER (width, n); + } #endif case L_('n'): |