Skip to content

Commit 59fdb34

Browse files
adetayloradetaylor
committed
Avoid GNU folding constant warnings
https://bugs.webkit.org/show_bug.cgi?id=297660 rdar://158772091 Reviewed by Justin Michaud. Adds extra warning suppressions which may be needed by different toolchains. * Source/JavaScriptCore/API/tests/testapi.c: (checkJSStringOOBUTF8): (checkJSStringOOBUTF16): (checkJSStringOOBUTF16AtEnd): Canonical link: https://commits.webkit.org/299007@main
1 parent f5332f0 commit 59fdb34

1 file changed

Lines changed: 12 additions & 0 deletions

File tree

‎Source/JavaScriptCore/API/tests/testapi.c‎

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1196,14 +1196,18 @@ static void checkJSStringOOBUTF8(void)
11961196
const size_t outCStringSize = cStringSize + sourceCStringSize;
11971197

11981198
IGNORE_WARNINGS_BEGIN("vla")
1199+
IGNORE_WARNINGS_BEGIN("gnu-folding-constant")
11991200
char sourceCString[sourceCStringSize];
1201+
IGNORE_WARNINGS_END
12001202
IGNORE_WARNINGS_END
12011203
memset(sourceCString, 0, sizeof(sourceCString));
12021204
for (size_t i = 0; i < sourceCStringSize - 1; ++i)
12031205
sourceCString[i] = '0' + (i%10);
12041206

12051207
IGNORE_WARNINGS_BEGIN("vla")
1208+
IGNORE_WARNINGS_BEGIN("gnu-folding-constant")
12061209
char outCString[outCStringSize];
1210+
IGNORE_WARNINGS_END
12071211
IGNORE_WARNINGS_END
12081212
memset(outCString, 0x13, sizeof(outCString));
12091213

@@ -1231,7 +1235,9 @@ static void checkJSStringOOBUTF16(void)
12311235
const size_t outCStringSize = cStringSize + sourceCStringSize;
12321236

12331237
IGNORE_WARNINGS_BEGIN("vla")
1238+
IGNORE_WARNINGS_BEGIN("gnu-folding-constant")
12341239
char sourceCString[sourceCStringSize];
1240+
IGNORE_WARNINGS_END
12351241
IGNORE_WARNINGS_END
12361242
memset(sourceCString, 0, sizeof(sourceCString));
12371243
for (size_t i = 0; i < sourceCStringSize - 1; ++i)
@@ -1243,7 +1249,9 @@ IGNORE_WARNINGS_END
12431249
sourceCString[6] = '\x81';
12441250

12451251
IGNORE_WARNINGS_BEGIN("vla")
1252+
IGNORE_WARNINGS_BEGIN("gnu-folding-constant")
12461253
char outCString[outCStringSize];
1254+
IGNORE_WARNINGS_END
12471255
IGNORE_WARNINGS_END
12481256
memset(outCString, 0x13, sizeof(outCString));
12491257

@@ -1271,7 +1279,9 @@ static void checkJSStringOOBUTF16AtEnd(void)
12711279
const size_t outCStringSize = cStringSize + sourceCStringSize;
12721280

12731281
IGNORE_WARNINGS_BEGIN("vla")
1282+
IGNORE_WARNINGS_BEGIN("gnu-folding-constant")
12741283
char sourceCString[sourceCStringSize];
1284+
IGNORE_WARNINGS_END
12751285
IGNORE_WARNINGS_END
12761286
memset(sourceCString, 0, sizeof(sourceCString));
12771287
for (size_t i = 0; i < sourceCStringSize - 1; ++i)
@@ -1283,7 +1293,9 @@ IGNORE_WARNINGS_END
12831293
sourceCString[20] = '\x81';
12841294

12851295
IGNORE_WARNINGS_BEGIN("vla")
1296+
IGNORE_WARNINGS_BEGIN("gnu-folding-constant")
12861297
char outCString[outCStringSize];
1298+
IGNORE_WARNINGS_END
12871299
IGNORE_WARNINGS_END
12881300
memset(outCString, 0x13, sizeof(outCString));
12891301

0 commit comments

Comments
 (0)