[PATCH] D90116: [clangd] Escape Unicode characters to fix Windows builds
Kirill Bobyrev via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Sun Oct 25 03:58:57 PDT 2020
kbobyrev created this revision.
kbobyrev added reviewers: kadircet, sammccall.
Herald added subscribers: cfe-commits, usaxena95, arphaman.
Herald added a project: clang.
kbobyrev requested review of this revision.
Herald added subscribers: MaskRay, ilya-biryukov.
Fixes: https://github.com/clangd/clangd/issues/571
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D90116
Files:
clang-tools-extra/clangd/CodeComplete.h
clang-tools-extra/clangd/Diagnostics.cpp
Index: clang-tools-extra/clangd/Diagnostics.cpp
===================================================================
--- clang-tools-extra/clangd/Diagnostics.cpp
+++ clang-tools-extra/clangd/Diagnostics.cpp
@@ -545,7 +545,7 @@
OS << R;
if (R.size() != Code.size())
- OS << "…";
+ OS << "\u2026"; // Unicode Horizontal Ellipsis.
}
/// Fills \p D with all information, except the location-related bits.
Index: clang-tools-extra/clangd/CodeComplete.h
===================================================================
--- clang-tools-extra/clangd/CodeComplete.h
+++ clang-tools-extra/clangd/CodeComplete.h
@@ -86,7 +86,7 @@
/// A visual indicator to prepend to the completion label to indicate whether
/// completion result would trigger an #include insertion or not.
struct IncludeInsertionIndicator {
- std::string Insert = "•";
+ std::string Insert = "\u2022"; // Unicode Bullet.
std::string NoInsert = " ";
} IncludeIndicator;
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D90116.300534.patch
Type: text/x-patch
Size: 975 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20201025/3806325a/attachment.bin>
More information about the cfe-commits
mailing list