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

Simon Pilgrim via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Oct 25 09:30:31 PDT 2022


RKSimon added inline comments.


================
Comment at: llvm/utils/UnicodeData/UnicodeNameMappingGenerator.cpp:46
+      auto SecondSemiPos = Line.find(';', FirstSemiPos + 1);
+      if (FirstSemiPos == std::string::npos)
+        continue;
----------------
@cor3ntin Should this be SecondSemiPos ?

Report here: https://pvs-studio.com/en/blog/posts/cpp/1003/ (N39)


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