Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 # Copyright (c) 2012 The Chromium Authors. All rights reserved. | 1 # Copyright (c) 2012 The Chromium Authors. All rights reserved. |
| 2 # Use of this source code is governed by a BSD-style license that can be | 2 # Use of this source code is governed by a BSD-style license that can be |
| 3 # found in the LICENSE file. | 3 # found in the LICENSE file. |
| 4 | 4 |
| 5 """Top-level presubmit script for cc. | 5 """Top-level presubmit script for cc. |
| 6 | 6 |
| 7 See http://dev.chromium.org/developers/how-tos/depottools/presubmit-scripts | 7 See http://dev.chromium.org/developers/how-tos/depottools/presubmit-scripts |
| 8 for more details about the presubmit API built into depot_tools. | 8 for more details about the presubmit API built into depot_tools. |
| 9 """ | 9 """ |
| 10 | 10 |
| 11 import re | 11 import re |
| 12 import string | 12 import string |
| 13 | 13 |
| 14 CC_SOURCE_FILES=(r'^cc[\\/].*\.(cc|h)$',) | 14 CC_SOURCE_FILES=(r'^cc[\\/].*\.(cc|h)$',) |
| 15 | 15 |
| 16 def CheckChangeLintsClean(input_api, output_api): | 16 def CheckChangeLintsClean(input_api, output_api): |
| 17 input_api.cpplint._cpplint_state.ResetErrorCounts() # reset global state | 17 input_api.cpplint._cpplint_state.ResetErrorCounts() # reset global state |
| 18 source_filter = lambda x: input_api.FilterSourceFile( | 18 source_filter = lambda x: input_api.FilterSourceFile( |
| 19 x, white_list=CC_SOURCE_FILES, black_list=None) | 19 x, white_list=CC_SOURCE_FILES, black_list=None) |
| 20 files = [f.AbsoluteLocalPath() for f in | |
|
tfarina
2015/02/24 01:50:23
This code is basically repeating what CheckChangeL
| |
| 21 input_api.AffectedSourceFiles(source_filter)] | |
| 22 level = 1 # strict, but just warn | |
| 23 | 20 |
| 24 for file_name in files: | 21 return input_api.canned_checks.CheckChangeLintsClean( |
|
tfarina
2015/02/24 01:50:24
Is this OK?
| |
| 25 input_api.cpplint.ProcessFile(file_name, level) | 22 input_api, output_api, source_filter, []) |
| 26 | |
| 27 if not input_api.cpplint._cpplint_state.error_count: | |
| 28 return [] | |
| 29 | |
| 30 return [output_api.PresubmitPromptWarning( | |
| 31 'Changelist failed cpplint.py check.')] | |
| 32 | 23 |
| 33 def CheckAsserts(input_api, output_api, white_list=CC_SOURCE_FILES, black_list=N one): | 24 def CheckAsserts(input_api, output_api, white_list=CC_SOURCE_FILES, black_list=N one): |
| 34 black_list = tuple(black_list or input_api.DEFAULT_BLACK_LIST) | 25 black_list = tuple(black_list or input_api.DEFAULT_BLACK_LIST) |
| 35 source_file_filter = lambda x: input_api.FilterSourceFile(x, white_list, black _list) | 26 source_file_filter = lambda x: input_api.FilterSourceFile(x, white_list, black _list) |
| 36 | 27 |
| 37 assert_files = [] | 28 assert_files = [] |
| 38 notreached_files = [] | 29 notreached_files = [] |
| 39 | 30 |
| 40 for f in input_api.AffectedSourceFiles(source_file_filter): | 31 for f in input_api.AffectedSourceFiles(source_file_filter): |
| 41 contents = input_api.ReadFile(f, 'rb') | 32 contents = input_api.ReadFile(f, 'rb') |
| (...skipping 294 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 336 results += FindUselessIfdefs(input_api, output_api) | 327 results += FindUselessIfdefs(input_api, output_api) |
| 337 results += input_api.canned_checks.CheckPatchFormatted(input_api, output_api) | 328 results += input_api.canned_checks.CheckPatchFormatted(input_api, output_api) |
| 338 return results | 329 return results |
| 339 | 330 |
| 340 def GetPreferredTryMasters(project, change): | 331 def GetPreferredTryMasters(project, change): |
| 341 return { | 332 return { |
| 342 'tryserver.blink': { | 333 'tryserver.blink': { |
| 343 'linux_blink_rel': set(['defaulttests']), | 334 'linux_blink_rel': set(['defaulttests']), |
| 344 }, | 335 }, |
| 345 } | 336 } |
| OLD | NEW |