[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 06:42:06 PDT 2020


kbobyrev updated this revision to Diff 300535.
kbobyrev added a comment.

Use u8 string literals.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D90116/new/

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 << u8"\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 = u8"\u2022"; // Unicode Bullet.
     std::string NoInsert = " ";
   } IncludeIndicator;
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D90116.300535.patch
Type: text/x-patch
Size: 979 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20201025/dcb1ea90/attachment.bin>


More information about the cfe-commits mailing list