[libcxx-commits] [libcxx] [libcxx] [regex] add `[[clang::fallthrough]]` to suppress fallthrough warning (PR #100821)

Max Coplan via libcxx-commits libcxx-commits at lists.llvm.org
Sun Jul 28 14:03:26 PDT 2024


================
@@ -3921,7 +3921,7 @@ _ForwardIterator basic_regex<_CharT, _Traits>::__parse_character_escape(
       if (__hd == -1)
         __throw_regex_error<regex_constants::error_escape>();
       __sum = 16 * __sum + static_cast<unsigned>(__hd);
-      // fallthrough
+    [[clang::fallthrough]];
----------------
vegerot wrote:

1. done
2. Whether I put a `break;` or fallthrough here, the tests pass which is a little sus. (thanks to @philnik777 for [helping](https://discord.com/channels/636084430946959380/636732894974312448/1267222316967002244) me run the tests)

https://github.com/llvm/llvm-project/pull/100821


More information about the libcxx-commits mailing list