[PATCH 15/59] iconvdata: Fix clang -Wstring-plus-int clang warning
Adhemerval Zanella
adhemerval.zanella@linaro.org
Fri Oct 17 19:12:50 GMT 2025
clang issues an warning adding '{unsigned} int' to a string does not
append to the string.
Use array indexes instead of string addition (it is simpler than
add a supress warning)
---
iconvdata/gbk.c | 6 +++---
iconvdata/iso-2022-cn-ext.c | 4 ++--
iconvdata/iso-2022-cn.c | 2 +-
3 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/iconvdata/gbk.c b/iconvdata/gbk.c
index ac77c891ce..54ddcc079c 100644
--- a/iconvdata/gbk.c
+++ b/iconvdata/gbk.c
@@ -13286,7 +13286,7 @@ static const char __gbk_from_ucs4_tab12[][2] =
cp = "\xa8\xc0"; \
break; \
case 0x2c7 ... 0x2cb: \
- cp = "\xa1\xa6\0\0\0\0\0\0\xa1\xa5\0\0\xa8\x40\0\0\xa8\x41" + ((ch - 0x2c7) * 4); \
+ cp = &"\xa1\xa6\0\0\0\0\0\0\xa1\xa5\0\0\xa8\x40\0\0\xa8\x41"[(ch - 0x2c7) * 4]; \
break; \
case 0x2d9: \
cp = "\xa8\x42"; \
@@ -13330,7 +13330,7 @@ static const char __gbk_from_ucs4_tab12[][2] =
buf[1] = '\x80' + (ch - 0x2588); \
break; \
case 0x2593 ... 0x2595: \
- cp = "\xa8\x88\0\0\xa8\x89\0\0\xa8\x8a" + ((ch - 0x2593) * 4); \
+ cp = &"\xa8\x88\0\0\xa8\x89\0\0\xa8\x8a"[(ch - 0x2593) * 4]; \
break; \
case 0x25a0: \
cp = "\xa1\xf6"; \
@@ -13461,7 +13461,7 @@ static const char __gbk_from_ucs4_tab12[][2] =
cp = __gbk_from_ucs4_tab12[ch - 0xff01]; \
break; \
case 0xffe0 ... 0xffe5: \
- cp = "\xa1\xe9\0\0\xa1\xea\0\0\xa9\x56\0\0\xa3\xfe\0\0\xa9\x57\0\0\xa3\xa4" + ((ch - 0xffe0) * 4); \
+ cp = &"\xa1\xe9\0\0\xa1\xea\0\0\xa9\x56\0\0\xa3\xfe\0\0\xa9\x57\0\0\xa3\xa4" [(ch - 0xffe0) * 4]; \
break; \
default: \
UNICODE_TAG_HANDLER (ch, 4); \
diff --git a/iconvdata/iso-2022-cn-ext.c b/iconvdata/iso-2022-cn-ext.c
index 2afa5e3b4d..ade40c745e 100644
--- a/iconvdata/iso-2022-cn-ext.c
+++ b/iconvdata/iso-2022-cn-ext.c
@@ -562,7 +562,7 @@ DIAG_IGNORE_Os_NEEDS_COMMENT_GCC (5, "-Wmaybe-uninitialized");
} \
\
assert (used >= 1 && used <= 4); \
- escseq = ")A\0\0)G)E" + (used - 1) * 2; \
+ escseq = &")A\0\0)G)E"[(used - 1) * 2]; \
*outptr++ = ESC; \
*outptr++ = '$'; \
*outptr++ = *escseq++; \
@@ -600,7 +600,7 @@ DIAG_IGNORE_Os_NEEDS_COMMENT_GCC (5, "-Wmaybe-uninitialized");
} \
\
assert ((used >> 5) >= 3 && (used >> 5) <= 7); \
- escseq = "+I+J+K+L+M" + ((used >> 5) - 3) * 2; \
+ escseq = &"+I+J+K+L+M"[((used >> 5) - 3) * 2]; \
*outptr++ = ESC; \
*outptr++ = '$'; \
*outptr++ = *escseq++; \
diff --git a/iconvdata/iso-2022-cn.c b/iconvdata/iso-2022-cn.c
index 0b81714256..7fe4b36f27 100644
--- a/iconvdata/iso-2022-cn.c
+++ b/iconvdata/iso-2022-cn.c
@@ -326,7 +326,7 @@ enum
} \
\
assert ((used >> 3) >= 1 && (used >> 3) <= 3); \
- escseq = ")A)G*H" + ((used >> 3) - 1) * 2; \
+ escseq = &")A)G*H"[((used >> 3) - 1) * 2]; \
*outptr++ = ESC; \
*outptr++ = '$'; \
*outptr++ = *escseq++; \
--
2.43.0
More information about the Libc-alpha
mailing list