chaoren added inline comments.
================
Comment at: test/dosep.py:163
@@ -162,3 @@
- # We're only interested in the test file with the "Test*.py" naming
pattern.
- if not name.startswith("Test") or not name.endswith(".py"):
- continue
----------------
From here.
================
Comment at: test/dosep.py:215
@@ +214,3 @@
+ # Only interested in test files with the "Test*.py" naming pattern.
+ return name.startswith("Test") and name.endswith(".py")
+
----------------
zturner wrote:
> How was this working before? Seems risky to start eliminating files that
> were previously considered, and is this rule documented anywhere that the
> filename must start with Test?
Yeah, this is moved over from line 163. I just moved the filtering earlier to
get a count of test suites.
http://reviews.llvm.org/D11843
_______________________________________________
lldb-commits mailing list
[email protected]
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
