diff options
author | Jason A. Donenfeld <Jason@zx2c4.com> | 2022-11-03 01:50:05 +0100 |
---|---|---|
committer | Jason A. Donenfeld <Jason@zx2c4.com> | 2022-11-19 00:56:15 +0100 |
commit | 0445d1bae1cce00ae4e99c8cde33784a8199bad6 (patch) | |
tree | 6170247bbe5e117550440b6d508a520606cbc245 | |
parent | 3bc753c06dd02a3517c9b498e3846ebfc94ac3ee (diff) | |
download | linux-stable-0445d1bae1cce00ae4e99c8cde33784a8199bad6.tar.gz linux-stable-0445d1bae1cce00ae4e99c8cde33784a8199bad6.tar.bz2 linux-stable-0445d1bae1cce00ae4e99c8cde33784a8199bad6.zip |
lib: assume char is unsigned
Now that we use -funsigned-char, there's no need for this kind of ifdef.
Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
-rw-r--r-- | lib/is_signed_type_kunit.c | 4 | ||||
-rw-r--r-- | lib/test_printf.c | 12 |
2 files changed, 0 insertions, 16 deletions
diff --git a/lib/is_signed_type_kunit.c b/lib/is_signed_type_kunit.c index 207207522925..0a7f6ae62839 100644 --- a/lib/is_signed_type_kunit.c +++ b/lib/is_signed_type_kunit.c @@ -21,11 +21,7 @@ static void is_signed_type_test(struct kunit *test) KUNIT_EXPECT_EQ(test, is_signed_type(bool), false); KUNIT_EXPECT_EQ(test, is_signed_type(signed char), true); KUNIT_EXPECT_EQ(test, is_signed_type(unsigned char), false); -#ifdef __CHAR_UNSIGNED__ KUNIT_EXPECT_EQ(test, is_signed_type(char), false); -#else - KUNIT_EXPECT_EQ(test, is_signed_type(char), true); -#endif KUNIT_EXPECT_EQ(test, is_signed_type(int), true); KUNIT_EXPECT_EQ(test, is_signed_type(unsigned int), false); KUNIT_EXPECT_EQ(test, is_signed_type(long), true); diff --git a/lib/test_printf.c b/lib/test_printf.c index 4bd15a593fbd..5eb889679e4f 100644 --- a/lib/test_printf.c +++ b/lib/test_printf.c @@ -179,18 +179,6 @@ test_number(void) * behaviour. */ test("00|0|0|0|0", "%.2d|%.1d|%.0d|%.*d|%1.0d", 0, 0, 0, 0, 0, 0); -#ifndef __CHAR_UNSIGNED__ - { - /* - * Passing a 'char' to a %02x specifier doesn't do - * what was presumably the intention when char is - * signed and the value is negative. One must either & - * with 0xff or cast to u8. - */ - char val = -16; - test("0xfffffff0|0xf0|0xf0", "%#02x|%#02x|%#02x", val, val & 0xff, (u8)val); - } -#endif } static void __init |