Ilia K ki.stfu at gmail.com
Wed Jul 29 22:14:08 PDT 2015

ki.stfu added inline comments.

================
@@ -33,1 +31,3 @@
+            ])

----------------
ChuckR wrote:
> ki.stfu wrote:
> > it may cause false negative on systems where full path to lldb dir contains bad symbols that should be escaped (" or \ etc). I had the same issue in MiSyntaxTestCase.
> Any suggestions for checking the size then? I could make a second test that only checks for the size field?
I think u can use an analogue of AddSlashes:

>>> def add_slashes(x): return x.replace("\\", "\\\\").replace("\"", "\\\"").replace("\'", "\\\'")
...
>>> print "C:\\foo\\bar\\baz\"\'ext"
C:\foo\bar\baz"'ext
C:\\foo\\bar\\baz\"\'ext


http://reviews.llvm.org/D9716