On 28/02/2019 02:37, Davide Italiano via Phabricator wrote:
davide added a comment.
Herald added a subscriber: jdoerfert.
Herald added a project: LLVM.

Pavel, this broke unicode handling for lldb on MacOS. If you type something in 
the lldb cmdline, it won't print the right character but a series of unicode 
sequences.
The main concern is that this also breaks the Swift REPL (powered by lldb).

Looks like something like this:

   $ git diff
   diff --git a/lldb/include/lldb/Host/Editline.h 
b/lldb/include/lldb/Host/Editline.h
   index cb40f1e6917..e53991c450f 100644
   --- a/lldb/include/lldb/Host/Editline.h
   +++ b/lldb/include/lldb/Host/Editline.h
   @@ -51,6 +51,7 @@
    #include <string>
    #include <vector>
+#include "lldb/Host/Config.h"
    #include "lldb/Host/ConnectionFileDescriptor.h"
    #include "lldb/Host/Predicate.h"
    #include "lldb/Utility/FileSpec.h"

"fixes" the CMake build. I'm seeing failures if I apply the same patch to the 
Xcode project.

Have you tried putting the `#include "lldb/Host/Config.h"` at the top of Editline.h, so that LLDB_EDITLINE_USE_WCHAR is defined before the
```
#if LLDB_EDITLINE_USE_WCHAR
#include <codecvt>
#endif
```
block ?
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
  • [Lldb-commits] [PATCH] D4... Davide Italiano via Phabricator via lldb-commits
    • Re: [Lldb-commits] [... Pavel Labath via lldb-commits
      • Re: [Lldb-commit... Pavel Labath via lldb-commits
        • Re: [Lldb-co... Raphael Isemann via lldb-commits
          • Re: [Lld... Davide Italiano via lldb-commits
            • Re:... Davide Italiano via lldb-commits
              • ... Davide Italiano via lldb-commits
                • ... Davide Italiano via lldb-commits
                • ... Davide Italiano via lldb-commits
                • ... Pavel Labath via lldb-commits

Reply via email to