[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