Skip to content

Commit d09f18e

Browse files
committed
formatting + explicit casts
1 parent 14fe00a commit d09f18e

File tree

1 file changed

+15
-5
lines changed

1 file changed

+15
-5
lines changed

libc/test/src/__support/wchar/string_converter_test.cpp

Lines changed: 15 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,10 @@ TEST(LlvmLibcStringConverterTest, UTF8To32) {
5757

5858
TEST(LlvmLibcStringConverterTest, UTF32To8) {
5959
// clown emoji, sigma symbol, y with diaeresis, letter A
60-
const wchar_t src[] = {0x1f921, 0x2211, 0xff, 0x41, 0x0};
60+
const wchar_t src[] = {static_cast<wchar_t>(0x1f921),
61+
static_cast<wchar_t>(0x2211),
62+
static_cast<wchar_t>(0xff), static_cast<wchar_t>(0x41),
63+
static_cast<wchar_t>(0x0)};
6164
LIBC_NAMESPACE::internal::mbstate state;
6265
LIBC_NAMESPACE::internal::StringConverter<char32_t> sc(
6366
reinterpret_cast<const char32_t *>(src), &state, SIZE_MAX);
@@ -128,7 +131,9 @@ TEST(LlvmLibcStringConverterTest, UTF32To8) {
128131
}
129132

130133
TEST(LlvmLibcStringConverterTest, UTF32To8PartialRead) {
131-
const wchar_t src[] = {0x1f921, 0x2211, 0x0}; // clown emoji, sigma symbol
134+
const wchar_t src[] = {
135+
static_cast<wchar_t>(0x1f921), static_cast<wchar_t>(0x2211),
136+
static_cast<wchar_t>(0x0)}; // clown emoji, sigma symbol
132137
LIBC_NAMESPACE::internal::mbstate state;
133138
LIBC_NAMESPACE::internal::StringConverter<char32_t> sc(
134139
reinterpret_cast<const char32_t *>(src), &state, SIZE_MAX, 1);
@@ -178,7 +183,9 @@ TEST(LlvmLibcStringConverterTest, UTF8To32PartialRead) {
178183
}
179184

180185
TEST(LlvmLibcStringConverterTest, UTF32To8ErrorHandling) {
181-
const wchar_t src[] = {0x1f921, 0xffffff, 0x0}; // clown emoji, invalid utf32
186+
const wchar_t src[] = {
187+
static_cast<wchar_t>(0x1f921), static_cast<wchar_t>(0xffffff),
188+
static_cast<wchar_t>(0x0)}; // clown emoji, invalid utf32
182189
LIBC_NAMESPACE::internal::mbstate state;
183190
LIBC_NAMESPACE::internal::StringConverter<char32_t> sc(
184191
reinterpret_cast<const char32_t *>(src), &state, SIZE_MAX);
@@ -234,7 +241,9 @@ TEST(LlvmLibcStringConverterTest, MultipleStringConverters32To8) {
234241
StringConverter to continue where we left off. This is not expected to work
235242
and considered invalid.
236243
*/
237-
const wchar_t src[] = {0x1f921, 0xff, 0x0}; // clown emoji, y with diaeresis (ÿ)
244+
const wchar_t src[] = {
245+
static_cast<wchar_t>(0x1f921), static_cast<wchar_t>(0xff),
246+
static_cast<wchar_t>(0x0)}; // clown emoji, y with diaeresis (ÿ)
238247
LIBC_NAMESPACE::internal::mbstate state;
239248
LIBC_NAMESPACE::internal::StringConverter<char32_t> sc1(
240249
reinterpret_cast<const char32_t *>(src), &state, SIZE_MAX, 1);
@@ -317,7 +326,8 @@ TEST(LlvmLibcStringConverterTest, DestLimitUTF8To32) {
317326
}
318327

319328
TEST(LlvmLibcStringConverterTest, DestLimitUTF32To8) {
320-
const wchar_t src[] = {0x1f921, 0x1f921}; // 2 clown emojis
329+
const wchar_t src[] = {static_cast<wchar_t>(0x1f921),
330+
static_cast<wchar_t>(0x1f921)}; // 2 clown emojis
321331
LIBC_NAMESPACE::internal::mbstate state;
322332
LIBC_NAMESPACE::internal::StringConverter<char32_t> sc(
323333
reinterpret_cast<const char32_t *>(src), &state, 5);

0 commit comments

Comments
 (0)