ovyalov added a comment.

In http://reviews.llvm.org/D20548#437567, @labath wrote:

> According to 
> https://msdn.microsoft.com/en-gb/library/windows/desktop/aa365247%28v=vs.85%29.aspx?f=255&MSPPError=-2147217396,
>  characters with codes 1..31 (basically `< ' '`), are illegal as well. We 
> might as well convert those, just in case.


Agree - done.

> I've been sitting on a unit test for the module cache locally for a while 
> now... I guess it's time to put that in, and then we can add a test for this 
> as well.


Sounds good.


http://reviews.llvm.org/D20548



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to