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

Thank you.  This looks good, assuming that the LLDB_DOTEST_ARGS_STR thingy is 
working as intended.



================
Comment at: test/lldb-dotest.in:6
 dotest_path = '@LLDB_SOURCE_DIR@/test/dotest.py'
-dotest_args = '@LLDB_DOTEST_ARGS_STR@'
+dotest_args_str = '@LLDB_DOTEST_ARGS_STR@'
 
----------------
I'm confused. Shouldn't this be `@LLDB_DOTEST_ARGS@`, now that you've removed 
the _STR version?


https://reviews.llvm.org/D44728



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

Reply via email to