[PATCH] D29039: [clang-format] Proposal for clang-format -r option

MyDeveloperDay via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Oct 10 00:11:30 PDT 2019


MyDeveloperDay added a comment.





================
Comment at: tools/clang-format/ClangFormat.cpp:345
+
+      Regex regex(FileNames[i]);
+
----------------
I could imagine the path you supplying being just a directory . or  ../../include/Format as much as a wild card, once in that directory I might want to you clang-format all the file types clang-format supports or just a few? how could we do that?

it would be nice if we could supply multiple directories on the command line

```clang-format -r -i  . .. ../include/Format```

I wonder if the wild card should act more like --gtest_filter   

```--gtest_filter=POSTIVE_PATTERNS[-NEGATIVE_PATTERNS]``` 

or find's -iregex?




Repository:
  rL LLVM

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

https://reviews.llvm.org/D29039





More information about the cfe-commits mailing list