labath accepted this revision.
labath added a comment.
This revision is now accepted and ready to land.

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.

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.


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