ClockMan added inline comments.
================
Comment at: clang-tools-extra/clang-tidy/tool/ClangTidyMain.cpp:34
+static StringRef TrimFirstChar(StringRef x) { return x.substr(1); }
+
----------------
Looks like usecase of this is more like rtrim, then probably rtrim should be
used instead of substr.
And probably better would be to replace entire cl::desc. instead of adding new
additional wrap.
more like:
static auto desc(llvm::StringRef description) { return
cl::desc(description.rtrim()); }
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D141144/new/
https://reviews.llvm.org/D141144
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
