HazardyKnusperkeks 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 ---------------- MyDeveloperDay wrote: > 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. Maybe, but I doubt we want http. And if we use `file://` a windows path `D:\Path\` would not be valid, would it? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72326/new/ https://reviews.llvm.org/D72326 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
