Commit 260322d3 authored by Jan Wilken Dörrie's avatar Jan Wilken Dörrie Committed by Chromium LUCI CQ

[clang-tidy] Remove PRESUBMIT check for NULL

This change removes the PRESUBMIT check for NULL, since it is rendered
obsolete by the modernize-use-nullptr clang-tidy checked introduced in
r709137. The latter check produces less false positives, since it only
runs on changed lines, rather than across changed files.

Bug: None
Change-Id: I0ee29c95e1e53a6f421377932a85edff1466b6b1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2584162
Commit-Queue: Jan Wilken Dörrie <jdoerrie@chromium.org>
Reviewed-by: default avatarDaniel Cheng <dcheng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#836027}
parent 15d43aa6
...@@ -399,14 +399,6 @@ _NOT_CONVERTED_TO_MODERN_BIND_AND_CALLBACK = '|'.join(( ...@@ -399,14 +399,6 @@ _NOT_CONVERTED_TO_MODERN_BIND_AND_CALLBACK = '|'.join((
# * Error flag. True if a match is a presubmit error, otherwise it's a warning. # * Error flag. True if a match is a presubmit error, otherwise it's a warning.
# * Sequence of paths to *not* check (regexps). # * Sequence of paths to *not* check (regexps).
_BANNED_CPP_FUNCTIONS = ( _BANNED_CPP_FUNCTIONS = (
(
r'/\bNULL\b',
(
'New code should not use NULL. Use nullptr instead.',
),
False,
(),
),
( (
r'/\busing namespace ', r'/\busing namespace ',
( (
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment