[PATCH] D19031: [clang-format] Flexible line endings

MyDeveloperDay via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Nov 15 07:07:14 PST 2019


MyDeveloperDay accepted this revision.
MyDeveloperDay added a comment.
This revision is now accepted and ready to land.

This LGTM, I think this could help more than people might at first realize. I love the idea that I can use clang-format in dry-run (-n) mode to now make line endings seem a compile error..



================
Comment at: clang/lib/Format/Format.cpp:1360
+                        Style.UseCRLF) :
+          Style.UseCRLF);
     ContinuationIndenter Indenter(Style, Tokens.getKeywords(),
----------------
The great thing about this is if I don't want derived line endings I save scanning through the code twice counting the line endings


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

https://reviews.llvm.org/D19031





More information about the cfe-commits mailing list