[PATCH] D142804: [clang-format] Support clang-format on/off line comments as prefixes

Björn Schäpers via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Jan 28 04:19:22 PST 2023


HazardyKnusperkeks added inline comments.


================
Comment at: clang/lib/Format/Format.cpp:3901
+  return Comment.startswith(Prefix) &&
+         (Comment.size() == Size || isblank(Comment[Size]));
+}
----------------
rymiel wrote:
> Should the space be required? What about `// clang-format off: reasoning` or similar?
> 
> Also, should this be documented?
> Should the space be required? What about `// clang-format off: reasoning` or similar?
> 
> Also, should this be documented?

+1


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142804



More information about the cfe-commits mailing list