]> Savannah Git Hosting - gnulib.git/commitdiff
mbiterf: Fix compiler warnings.
authorBruno Haible <bruno@clisp.org>
Tue, 18 Jul 2023 11:04:04 +0000 (13:04 +0200)
committerBruno Haible <bruno@clisp.org>
Tue, 18 Jul 2023 11:04:04 +0000 (13:04 +0200)
* lib/mbiterf.h (mbiterf_next): Use C99 designated initializer syntax.
* tests/bench-mbiterf.c (do_test): Use a 'const char *' pointer.

ChangeLog
lib/mbiterf.h
tests/bench-mbiterf.c

index af0f128722df6307350080459fc55dd68cee7ca9..a4aa3b1714ace433387fabb371f8a82c4419e873 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,9 @@
 2023-07-18  Bruno Haible  <bruno@clisp.org>
 
+       mbiterf: Fix compiler warnings.
+       * lib/mbiterf.h (mbiterf_next): Use C99 designated initializer syntax.
+       * tests/bench-mbiterf.c (do_test): Use a 'const char *' pointer.
+
        trim: Optimize.
        * lib/trim.c: Include mbiterf.h instead of mbiter.h.
        (trim2): Use mbif_* macros instead of mbi_* macros.
index 0059c97fbdaef6789c83a1a5e10d7cd7fd351537..f5a817dbedb95e47139500750603faa888135a53 100644 (file)
@@ -120,7 +120,7 @@ mbiterf_next (struct mbif_state *ps, const char *iter, const char *endptr)
          their char code.  For the few other ones, this is the case as well,
          in all locale encodings that are in use.  The 32-bit wide character
          code is the same as well.  */
-      return (mbchar_t) { ptr: iter, bytes: 1, wc_valid: true, wc: *iter };
+      return (mbchar_t) { .ptr = iter, .bytes = 1, .wc_valid = true, .wc = *iter };
     }
   else
     {
@@ -140,7 +140,7 @@ mbiterf_next (struct mbif_state *ps, const char *iter, const char *endptr)
           ps->in_shift = false;
           #endif
           mbszero (&ps->state);
-          return (mbchar_t) { ptr: iter, bytes: 1, wc_valid: false };
+          return (mbchar_t) { .ptr = iter, .bytes = 1, .wc_valid = false };
         }
       else if (bytes == (size_t) -2)
         {
@@ -150,7 +150,7 @@ mbiterf_next (struct mbif_state *ps, const char *iter, const char *endptr)
           #endif
           /* Whether to reset ps->state or not is not important; the string end
              is reached anyway.  */
-          return (mbchar_t) { ptr: iter, bytes: endptr - iter, wc_valid: false };
+          return (mbchar_t) { .ptr = iter, .bytes = endptr - iter, .wc_valid = false };
         }
       else
         {
@@ -174,7 +174,7 @@ mbiterf_next (struct mbif_state *ps, const char *iter, const char *endptr)
           if (mbsinit (&ps->state))
             ps->in_shift = false;
           #endif
-          return (mbchar_t) { ptr: iter, bytes: bytes, wc_valid: true, wc: wc };
+          return (mbchar_t) { .ptr = iter, .bytes = bytes, .wc_valid = true, .wc = wc };
         }
     }
 }
index 0949fb6752f6dd76887faf80b1a3f3b414d867d1..bc3fbe4828f4f23b4fdbe525cff9cbd5a502f616 100644 (file)
@@ -43,7 +43,7 @@ do_test (char test, int repeat, const char *locale_name, const char *text)
           unsigned long long sum = 0;
           const char *text_end = text + text_len;
           mbif_state_t state;
-          char *iter;
+          const char *iter;
           for (mbif_init (state), iter = text; mbif_avail (state, iter, text_end); )
             {
               mbchar_t cur = mbif_next (state, iter, text_end);