Author: d0k Date: Tue Feb 7 10:10:17 2017 New Revision: 294314 URL: http://llvm.org/viewvc/llvm-project?rev=294314&view=rev Log: [clangd] Strip file:// from the URI when calling formatting.
It confuses FileManager on windows. Modified: clang-tools-extra/trunk/clangd/ProtocolHandlers.cpp Modified: clang-tools-extra/trunk/clangd/ProtocolHandlers.cpp URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/clangd/ProtocolHandlers.cpp?rev=294314&r1=294313&r2=294314&view=diff ============================================================================== --- clang-tools-extra/trunk/clangd/ProtocolHandlers.cpp (original) +++ clang-tools-extra/trunk/clangd/ProtocolHandlers.cpp Tue Feb 7 10:10:17 2017 @@ -63,6 +63,9 @@ static std::string formatCode(StringRef // Call clang-format. // FIXME: Don't ignore style. format::FormatStyle Style = format::getLLVMStyle(); + // On windows FileManager doesn't like file://. Just strip it, clang-format + // doesn't need it. + Filename.consume_front("file://"); tooling::Replacements Replacements = format::reformat(Style, Code, Ranges, Filename); _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits