test/message/testcfg.py - Issue 776563002: Better message location for 'super(...)' restriction error.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Unified Diff: test/message/testcfg.py

Issue 776563002: Better message location for 'super(...)' restriction error. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master

Patch Set: One more Created 6 years ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

View side-by-side diff with in-line comments

Download patch

Index: test/message/testcfg.py

diff --git a/test/message/testcfg.py b/test/message/testcfg.py

index b472f9cfb30b68ab5baf73545d1837b7d9746807..5d6ab8466396847aeb9040cd99e9095302d92761 100644

--- a/test/message/testcfg.py

+++ b/test/message/testcfg.py

@@ -75,6 +75,7 @@ class MessageTestSuite(testsuite.TestSuite):

def _IgnoreLine(self, string):

"""Ignore empty lines, valgrind output, Android output."""

if not string: return True

+ if not string.strip(): return True

Dmitry Lomov (no reviews) 2014/12/08 14:35:51 We strip expected too in line 94

return (string.startswith("==") or string.startswith("**") or

string.startswith("ANDROID") or

# These five patterns appear in normal Native Client output.

« no previous file with comments | « test/message/super-constructor-extra-statement.out ('k') | no next file » | no next file with comments »