Skip to content

Commit a6a39b0

Browse files
tkrusetheandrewdavis
tkruse
authored and
theandrewdavis
committed
Adapt linelength warnings in regression tests
1 parent 5225ba6 commit a6a39b0

6 files changed

Lines changed: 1079 additions & 2296 deletions

File tree

‎cpplint_clitest.py‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -114,6 +114,8 @@ def checkDef(self, path):
114114

115115
def runAndCheck(self, cwd, args, expectedStatus, expectedOut, expectedErr):
116116
cmd = BASE_CMD + ' ' + args
117+
# command to reproduce, do not forget first two lines have special meaning
118+
print("cd " + cwd + ";" + cmd + " 2> <filename>")
117119
(status, out, err) = RunShellCommand(cmd, cwd)
118120
try:
119121
self.assertEqual(expectedStatus, status)

‎samples/boost-sample/headers_simple.def‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -54,14 +54,14 @@ src/tr1/c_policy.hpp:69: Weird number of spaces at line-start. Are you using a
5454
src/tr1/c_policy.hpp:70: Weird number of spaces at line-start. Are you using a 2-space indent? [whitespace/indent] [3]
5555
src/tr1/c_policy.hpp:71: Weird number of spaces at line-start. Are you using a 2-space indent? [whitespace/indent] [3]
5656
src/tr1/c_policy.hpp:72: Weird number of spaces at line-start. Are you using a 2-space indent? [whitespace/indent] [3]
57-
src/tr1/c_policy.hpp:76: Lines should very rarely be longer than 100 characters [whitespace/line_length] [4]
57+
src/tr1/c_policy.hpp:76: Lines should be <= 80 characters long [whitespace/line_length] [2]
5858
src/tr1/c_policy.hpp:88: { should almost always be at the end of the previous line [whitespace/braces] [4]
5959
src/tr1/c_policy.hpp:89: Weird number of spaces at line-start. Are you using a 2-space indent? [whitespace/indent] [3]
60-
src/tr1/c_policy.hpp:89: Lines should very rarely be longer than 100 characters [whitespace/line_length] [4]
61-
src/tr1/c_policy.hpp:93: Lines should very rarely be longer than 100 characters [whitespace/line_length] [4]
60+
src/tr1/c_policy.hpp:89: Lines should be <= 80 characters long [whitespace/line_length] [2]
61+
src/tr1/c_policy.hpp:93: Lines should be <= 80 characters long [whitespace/line_length] [2]
6262
src/tr1/c_policy.hpp:105: { should almost always be at the end of the previous line [whitespace/braces] [4]
6363
src/tr1/c_policy.hpp:106: Weird number of spaces at line-start. Are you using a 2-space indent? [whitespace/indent] [3]
64-
src/tr1/c_policy.hpp:106: Lines should very rarely be longer than 100 characters [whitespace/line_length] [4]
64+
src/tr1/c_policy.hpp:106: Lines should be <= 80 characters long [whitespace/line_length] [2]
6565
src/tr1/c_policy.hpp:109: Namespace should be terminated with "// namespace policies" [readability/namespace] [5]
6666
src/tr1/c_policy.hpp:109: Namespace should be terminated with "// namespace math" [readability/namespace] [5]
6767
src/tr1/c_policy.hpp:109: Namespace should be terminated with "// namespace boost" [readability/namespace] [5]

0 commit comments

Comments
 (0)