ilya-biryukov added inline comments.
================
Comment at: clang/unittests/Tooling/TransformerTest.cpp:157
+ .as<clang::Expr>()
+ .replaceWith(text("REPLACED"))
+ .because(text("Use size() method directly on string."));
----------------
NIT: maybe consider adding overloads for these function that allow to pass char
literals there? This is probably a common case, so it'd be nice to write this
as:
```
replaceWith("REPLACED").because("Use size() method directly on string.");
```
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D59376/new/
https://reviews.llvm.org/D59376
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
