blob: d8fe08861bcd6a94b008c6ab851902c0fc1f807a [file] [log] [blame]
# Copyright 2015 The Chromium Authors. All rights reserved.
# Use of this source code is governed by a BSD-style license that can be
# found in the LICENSE file.
"""Presubmit script for ui.
See http://dev.chromium.org/developers/how-tos/depottools/presubmit-scripts
for more details about the presubmit API built into depot_tools.
"""
INCLUDE_CPP_FILES_ONLY = (
r'.*\.(cc|h|mm)$',
)
def CheckScopedPtr(input_api, output_api,
white_list=INCLUDE_CPP_FILES_ONLY, black_list=None):
black_list = tuple(black_list or input_api.DEFAULT_BLACK_LIST)
source_file_filter = lambda x: input_api.FilterSourceFile(x,
white_list,
black_list)
errors = []
for f in input_api.AffectedSourceFiles(source_file_filter):
for line_number, line in f.ChangedContents():
# Disallow:
# return scoped_ptr<T>(foo);
# bar = scoped_ptr<T>(foo);
# But allow:
# return scoped_ptr<T[]>(foo);
# bar = scoped_ptr<T[]>(foo);
if input_api.re.search(
r'(=|\breturn)\s*scoped_ptr<[^\[\]>]+>\([^)]+\)', line):
errors.append(output_api.PresubmitError(
('%s:%d uses explicit scoped_ptr constructor. ' +
'Use make_scoped_ptr() instead.') % (f.LocalPath(), line_number)))
# TODO(sky): this incorrectly catches templates. Fix and reenable.
# Disallow:
# scoped_ptr<T>()
# if input_api.re.search(r'\bscoped_ptr<.*?>\(\)', line):
# errors.append(output_api.PresubmitError(
# '%s:%d uses scoped_ptr<T>(). Use nullptr instead.' %
# (f.LocalPath(), line_number)))
return errors
def CheckChange(input_api, output_api):
results = []
results += CheckScopedPtr(input_api, output_api)
return results
def CheckChangeOnUpload(input_api, output_api):
return CheckChange(input_api, output_api)
def CheckChangeOnCommit(input_api, output_api):
return CheckChange(input_api, output_api)