]> Savannah Git Hosting - gnulib.git/commitdiff
posixtm: pacify clang 18
authorPaul Eggert <eggert@cs.ucla.edu>
Wed, 17 Jan 2024 01:21:08 +0000 (17:21 -0800)
committerBruno Haible <bruno@clisp.org>
Thu, 18 Jan 2024 08:34:36 +0000 (09:34 +0100)
Problem reported by Khem Raj in:
https://lists.gnu.org/r/bug-gnulib/2024-01/msg00045.html
* lib/posixtm.c (posixtime): Pacify clang 18 by converting bool to int.
Arguably this is a bug in draft C2x, since the non-pointer args to
ckd_add should promote just like any other expressions do;
but that’s not clang’s fault.

ChangeLog
lib/posixtm.c

index 512b97dd7bd3a87193904a7de06fb2620d3561d2..ba4ef25796fdd8852bfb44e9ee70fe95338c52a1 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,13 @@
+2024-01-16  Paul Eggert  <eggert@cs.ucla.edu>
+
+       posixtm: pacify clang 18
+       Problem reported by Khem Raj in:
+       https://lists.gnu.org/r/bug-gnulib/2024-01/msg00045.html
+       * lib/posixtm.c (posixtime): Pacify clang 18 by converting bool to int.
+       Arguably this is a bug in draft C2x, since the non-pointer args to
+       ckd_add should promote just like any other expressions do;
+       but that’s not clang’s fault.
+
 2024-01-15  Bruno Haible  <bruno@clisp.org>
 
        test-framework-sh: Improve portability to native Windows and OS/2.
index ef9f55f873ee71c63f33dab197d3187b899a5f29..a072c7cad015527ba548766b3bc2d47f67ab984b 100644 (file)
@@ -191,7 +191,7 @@ posixtime (time_t *p, const char *s, unsigned int syntax_bits)
              | (tm0.tm_min ^ tm1.tm_min)
              | (tm0.tm_sec ^ tm1.tm_sec)))
         {
-          if (ckd_add (&t, t, leapsec))
+          if (ckd_add (&t, t, +leapsec))
             return false;
           *p = t;
           return true;