[PATCH] D123064: [Clang][C++23] P2071 Named universal character escapes

Corentin Jabot via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Oct 25 09:39:55 PDT 2022


cor3ntin added inline comments.


================
Comment at: llvm/utils/UnicodeData/UnicodeNameMappingGenerator.cpp:46
+      auto SecondSemiPos = Line.find(';', FirstSemiPos + 1);
+      if (FirstSemiPos == std::string::npos)
+        continue;
----------------
RKSimon wrote:
> @cor3ntin Should this be SecondSemiPos ?
> 
> Report here: https://pvs-studio.com/en/blog/posts/cpp/1003/ (N39)
Yes. Nice catch.
There is no case of that happening in the file so it never manifested.
I'll push a commit fixing it.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D123064/new/

https://reviews.llvm.org/D123064



More information about the cfe-commits mailing list