<div dir="ltr">My impression is that the current state of LLDB with Unicode filenames and paths is pretty abysmal.  I assumed that nothing would work at all in the current codebase.  Are you saying that you are using LLDB to support Unicode today, or just that -- as a general matter - supporting Unicode strings is something we should do someday, so we shouldn't make our future work more difficult?<br></div><br><div class="gmail_quote">On Tue, Feb 24, 2015 at 9:34 PM <<a href="mailto:dawn@burble.org">dawn@burble.org</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Please keep in mind that we need to be able to handle Unicode strings<br>
and filenames, so please don't do anything that makes the handling of<br>
Unicode strings worse.  I.e. please don't add any (more) code which<br>
assumes a string is NULL-terminated.<br>
<br>
Thanks,<br>
-Dawn<br>
<br>
On Wed, Feb 11, 2015 at 07:52:26PM +0000, Zachary Turner wrote:<br>
> On Wed Feb 11 2015 at 11:13:22 AM Greg Clayton <<a href="mailto:gclayton@apple.com" target="_blank">gclayton@apple.com</a>> wrote:<br>
><br>
> ><br>
> > > Operations:<br>
> > > strcpy, strncpy, etc         // Use member functions of SmallString /<br>
> > StringRef<br>
> > > sprintf, snprintf, etc       // Use llvm::raw_ostream<br>
> ><br>
> > We have StringStream, people should be using that. raw_ostream doesn't<br>
> > support all printf style formats. So all sprintf  and snprintf function<br>
> > calls should switch to using lldb_private::StreamString.<br>
> ><br>
><br>
> What if llvm::SmallString<> had Format methods that delegated to snprintf<br>
> etc?  Then everything would just work, and it would integrate nicely with<br>
> the rest of my suggested points such as converting things to StringRef,<br>
> SmallVectorImpl, etc.<br>
<br>
> ______________________________<u></u>_________________<br>
> lldb-dev mailing list<br>
> <a href="mailto:lldb-dev@cs.uiuc.edu" target="_blank">lldb-dev@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev" target="_blank">http://lists.cs.uiuc.edu/<u></u>mailman/listinfo/lldb-dev</a><br>
<br>
</blockquote></div>