[PATCH] D72326: [clang-format] Rebased on master: Add option to specify explicit config file
MyDeveloperDay via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Nov 25 00:49:35 PST 2021
MyDeveloperDay added inline comments.
================
Comment at: clang/lib/Format/Format.cpp:3274
+ llvm::SmallVector<std::string, 2> FilesToLookFor;
+ // User provided clang-format file using -style=file:/path/to/format/file
+ // Check for explicit config filename
----------------
part of me wonders if this format should be
`file://<path>` rather than `file:<path>`
```User provided clang-format file using -style=file:///path/to/format/file```
vs
```User provided clang-format file using -style=file:/path/to/format/file```
This would leave the way open to other protocols http:// or https:// or anything else.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D72326/new/
https://reviews.llvm.org/D72326
More information about the cfe-commits
mailing list