Re: [patch] Fix parsing bug in validate_patches.py

[Diego Novillo]

On 2013-05-31 14:47 , Brooks Moses wrote:

Index: contrib/testsuite-management/validate_failures.py
===================================================================
--- contrib/testsuite-management/validate_failures.py    (revision 199390)
+++ contrib/testsuite-management/validate_failures.py    (working copy)
@@ -120,7 +120,7 @@
   def __init__(self, summary_line, ordinal=-1):
     try:
       self.attrs = ''
-      if '|' in summary_line:
+      if '|' in summary_line and not 
_VALID_TEST_RESULTS_REX.match(summary_line):

OK with this predicate factored into a predicate function (maybe 
SummaryLineHasAttributes?)


Thanks.  Diego.