[PATCH] D154093: [clang-format] Break long string literals in C#, etc.

Owen Pan via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Aug 24 22:03:18 PDT 2023


owenpan added a comment.

In D154093#4615948 <https://reviews.llvm.org/D154093#4615948>, @sstwcw wrote:

> In the JavaScript tests that I added, it was wrong to use `SmallString`.  Would you prefer me changing it to `string` or expanding the 6 test cases so we don't need a variable for the string?

You mean `std::string`? That would be fine by me.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D154093



More information about the cfe-commits mailing list