mirror of
https://git.freebsd.org/ports.git
synced 2025-05-21 03:23:10 -04:00
As noted in the libc++ 19 release notes [1], std::char_traits<> is now only provided for char, char8_t, char16_t, char32_t and wchar_t, and any instantiation for other types will fail. This causes ports using devel/rapidfuzz-cpp to fail to compile with clang 19 and libc++ 19, resulting in errors similar to: /usr/include/c++/v1/string:820:42: error: implicit instantiation of undefined template 'std::char_traits<unsigned int>' 820 | static_assert(is_same<_CharT, typename traits_type::char_type>::value, | ^ /wrkdirs/usr/ports/devel/py-rapidfuzz/work-py311/rapidfuzz-3.9.6/src/rapidfuzz/cpp_common.hpp:711:25: note: in instantiation of template class 'std::basic_string<unsigned int>' requested here 711 | auto proc_str = rf::opcodes_apply<uint32_t>(ops, s1, s2); | ^ The devel/rapidfuzz-cpp port itself does "build" since it only gathers a bunch of headers and installs them into the stage area, but running 'make test' also fails similarly: /usr/include/c++/v1/string:820:42: error: implicit instantiation of undefined template 'std::char_traits<unsigned char>' 820 | static_assert(is_same<_CharT, typename traits_type::char_type>::value, | ^ /wrkdirs/usr/ports/devel/rapidfuzz-cpp/work/rapidfuzz-cpp-3.0.5/test/distance/examples/ocr.cpp:3:28: note: in instantiation of template class 'std::basic_string<unsigned char>' requested here 3 | std::basic_string<uint8_t> ocr_example1 = { | ^ /usr/include/c++/v1/__fwd/string.h:23:29: note: template is declared here 23 | struct _LIBCPP_TEMPLATE_VIS char_traits; | ^ Unfortunately rapidfuzz-cpp makes heavy use of the no-longer-existing `std::basic_string<uint8_t>`, so I had to do quite a lots of search and replace operations, replacing these with equivalent `std::vector` types. Note that as far as I can see, only devel/py-rapidfuzz is a consumer of this port, applying these changes should not disrupt anything else. I have a follow-up patch for that port too. [1] https://libcxx.llvm.org/ReleaseNotes/19.html#deprecations-and-removals PR: 281193 Approved by: yuri (maintainer) MFH: 2024Q3
40 lines
1.7 KiB
C++
40 lines
1.7 KiB
C++
--- fuzzing/fuzz_levenshtein_distance.cpp.orig 2024-07-02 14:50:14 UTC
|
|
+++ fuzzing/fuzz_levenshtein_distance.cpp
|
|
@@ -9,7 +9,7 @@ template <size_t MaxLen>
|
|
#include <string>
|
|
|
|
template <size_t MaxLen>
|
|
-void validate_simd(const std::basic_string<uint8_t>& s1, const std::basic_string<uint8_t>& s2)
|
|
+void validate_simd(const std::vector<uint8_t>& s1, const std::vector<uint8_t>& s2)
|
|
{
|
|
#ifdef RAPIDFUZZ_SIMD
|
|
size_t count = s1.size() / MaxLen + ((s1.size() % MaxLen) != 0);
|
|
@@ -17,7 +17,7 @@ void validate_simd(const std::basic_string<uint8_t>& s
|
|
|
|
rapidfuzz::experimental::MultiLevenshtein<MaxLen> scorer(count);
|
|
|
|
- std::vector<std::basic_string<uint8_t>> strings;
|
|
+ std::vector<std::vector<uint8_t>> strings;
|
|
|
|
for (auto it1 = s1.begin(); it1 != s1.end(); it1 += MaxLen) {
|
|
if (std::distance(it1, s1.end()) < static_cast<ptrdiff_t>(MaxLen)) {
|
|
@@ -52,8 +52,8 @@ void validate_simd(const std::basic_string<uint8_t>& s
|
|
#endif
|
|
}
|
|
|
|
-void validate_distance(size_t reference_dist, const std::basic_string<uint8_t>& s1,
|
|
- const std::basic_string<uint8_t>& s2, size_t score_cutoff)
|
|
+void validate_distance(size_t reference_dist, const std::vector<uint8_t>& s1,
|
|
+ const std::vector<uint8_t>& s2, size_t score_cutoff)
|
|
{
|
|
if (reference_dist > score_cutoff) reference_dist = score_cutoff + 1;
|
|
|
|
@@ -75,7 +75,7 @@ extern "C" int LLVMFuzzerTestOneInput(const uint8_t* d
|
|
|
|
extern "C" int LLVMFuzzerTestOneInput(const uint8_t* data, size_t size)
|
|
{
|
|
- std::basic_string<uint8_t> s1, s2;
|
|
+ std::vector<uint8_t> s1, s2;
|
|
if (!extract_strings(data, size, s1, s2)) return 0;
|
|
|
|
size_t reference_dist = rapidfuzz_reference::levenshtein_distance(s1, s2);
|