diff options
-rw-r--r-- | meta-oe/recipes-dbs/leveldb/leveldb/0001-Use-static_cast-instead-of-ImplicitCast_-for-charact.patch | 49 | ||||
-rw-r--r-- | meta-oe/recipes-dbs/leveldb/leveldb_1.23.bb | 1 |
2 files changed, 50 insertions, 0 deletions
diff --git a/meta-oe/recipes-dbs/leveldb/leveldb/0001-Use-static_cast-instead-of-ImplicitCast_-for-charact.patch b/meta-oe/recipes-dbs/leveldb/leveldb/0001-Use-static_cast-instead-of-ImplicitCast_-for-charact.patch new file mode 100644 index 0000000000..7a06630918 --- /dev/null +++ b/meta-oe/recipes-dbs/leveldb/leveldb/0001-Use-static_cast-instead-of-ImplicitCast_-for-charact.patch | |||
@@ -0,0 +1,49 @@ | |||
1 | From fa8438ae6b70c57010177de47a9f13d7041a6328 Mon Sep 17 00:00:00 2001 | ||
2 | From: Abseil Team <absl-team@google.com> | ||
3 | Date: Mon, 19 May 2025 09:01:54 -0700 | ||
4 | Subject: [PATCH] Use static_cast instead of ImplicitCast_ for character | ||
5 | conversions | ||
6 | |||
7 | Clang has recently added "warnings when mixing different charN_t types" [1]. | ||
8 | The rationale is that "charN_t represent code units of different UTF encodings. | ||
9 | Therefore the values of 2 different charN_t objects do not represent the same | ||
10 | characters." | ||
11 | |||
12 | Note that the warning here may be legitimate - from https://github.com/google/googletest/issues/4762: | ||
13 | "[...] This is incorrect for values that do not represent valid codepoints." | ||
14 | |||
15 | For the time being, silence the warning by being more explicit about the | ||
16 | conversion being intentional by using static_cast. | ||
17 | |||
18 | Link: https://github.com/llvm/llvm-project/pull/138708 [1] | ||
19 | PiperOrigin-RevId: 760644157 | ||
20 | Change-Id: I2e6cc1871975455cecac8731b2f93fd5beeaf0e1 | ||
21 | |||
22 | Upstream-Status: Backport [https://github.com/google/googletest/commit/fa8438ae6b70c57010177de47a9f13d7041a6328] | ||
23 | Signed-off-by: Khem Raj <raj.khem@gmail.com> | ||
24 | --- | ||
25 | third_party/googletest/googletest/include/gtest/gtest-printers.h | 8 ++++++-- | ||
26 | 1 file changed, 6 insertions(+), 2 deletions(-) | ||
27 | |||
28 | Index: leveldb-1.23/third_party/googletest/googletest/include/gtest/gtest-printers.h | ||
29 | =================================================================== | ||
30 | --- leveldb-1.23.orig/third_party/googletest/googletest/include/gtest/gtest-printers.h | ||
31 | +++ leveldb-1.23/third_party/googletest/googletest/include/gtest/gtest-printers.h | ||
32 | @@ -471,11 +471,15 @@ GTEST_API_ void PrintTo(wchar_t wc, ::st | ||
33 | |||
34 | GTEST_API_ void PrintTo(char32_t c, ::std::ostream* os); | ||
35 | inline void PrintTo(char16_t c, ::std::ostream* os) { | ||
36 | - PrintTo(ImplicitCast_<char32_t>(c), os); | ||
37 | + // TODO(b/418738869): Incorrect for values not representing valid codepoints. | ||
38 | + // Also see https://github.com/google/googletest/issues/4762. | ||
39 | + PrintTo(static_cast<char32_t>(c), os); | ||
40 | } | ||
41 | #ifdef __cpp_char8_t | ||
42 | inline void PrintTo(char8_t c, ::std::ostream* os) { | ||
43 | - PrintTo(ImplicitCast_<char32_t>(c), os); | ||
44 | + // TODO(b/418738869): Incorrect for values not representing valid codepoints. | ||
45 | + // Also see https://github.com/google/googletest/issues/4762. | ||
46 | + PrintTo(static_cast<char32_t>(c), os); | ||
47 | } | ||
48 | #endif | ||
49 | |||
diff --git a/meta-oe/recipes-dbs/leveldb/leveldb_1.23.bb b/meta-oe/recipes-dbs/leveldb/leveldb_1.23.bb index 1ccf115793..90e0785e54 100644 --- a/meta-oe/recipes-dbs/leveldb/leveldb_1.23.bb +++ b/meta-oe/recipes-dbs/leveldb/leveldb_1.23.bb | |||
@@ -7,6 +7,7 @@ LIC_FILES_CHKSUM = "file://LICENSE;md5=92d1b128950b11ba8495b64938fc164d" | |||
7 | SRC_URI = "gitsm://github.com/google/${BPN}.git;branch=main;protocol=https \ | 7 | SRC_URI = "gitsm://github.com/google/${BPN}.git;branch=main;protocol=https \ |
8 | file://0001-CMakeLists.txt-fix-googletest-related-options.patch \ | 8 | file://0001-CMakeLists.txt-fix-googletest-related-options.patch \ |
9 | file://0001-Fix-printing-64-bit-integer-types.patch \ | 9 | file://0001-Fix-printing-64-bit-integer-types.patch \ |
10 | file://0001-Use-static_cast-instead-of-ImplicitCast_-for-charact.patch \ | ||
10 | file://run-ptest \ | 11 | file://run-ptest \ |
11 | " | 12 | " |
12 | 13 | ||