switch to 4 space indent
Leave the recipes/ code at 2 space to match the rest of the recipes
project in other repos.
Reformatted using:
files=( $(
git ls-tree -r --name-only HEAD | \
grep -Ev -e '^(third_party|recipes)/' | \
grep '\.py$';
git grep -l '#!/usr/bin/env.*python' | grep -v '\.py$'
) )
parallel ./yapf -i -- "${files[@]}"
~/chromiumos/chromite/contrib/reflow_overlong_comments "${files[@]}"
The files that still had strings that were too long were manually
reformatted because they were easy and only a few issues.
autoninja.py
clang_format.py
download_from_google_storage.py
fix_encoding.py
gclient_utils.py
git_cache.py
git_common.py
git_map_branches.py
git_reparent_branch.py
gn.py
my_activity.py
owners_finder.py
presubmit_canned_checks.py
reclient_helper.py
reclientreport.py
roll_dep.py
rustfmt.py
siso.py
split_cl.py
subcommand.py
subprocess2.py
swift_format.py
upload_to_google_storage.py
These files still had lines (strings) that were too long, so the pylint
warnings were suppressed with a TODO.
auth.py
gclient.py
gclient_eval.py
gclient_paths.py
gclient_scm.py
gerrit_util.py
git_cl.py
presubmit_canned_checks.py
presubmit_support.py
scm.py
Change-Id: Ia6535c4f2c48d46b589ec1e791dde6c6b2ea858f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/4836379
Commit-Queue: Josip Sokcevic <sokcevic@chromium.org>
Auto-Submit: Mike Frysinger <vapier@chromium.org>
Reviewed-by: Josip Sokcevic <sokcevic@chromium.org>
diff --git a/cpplint_chromium.py b/cpplint_chromium.py
index 9318a86..8604763 100755
--- a/cpplint_chromium.py
+++ b/cpplint_chromium.py
@@ -36,7 +36,7 @@
def CheckPointerDeclarationWhitespace(filename, clean_lines, linenum, error):
- """Checks for Foo *foo declarations.
+ """Checks for Foo *foo declarations.
Args:
filename: The name of the current file.
@@ -44,9 +44,10 @@
linenum: The number of the line to check.
error: The function to call with any errors found.
"""
- line = clean_lines.elided[linenum]
- matched = _RE_PATTERN_POINTER_DECLARATION_WHITESPACE.match(line)
- if matched:
- error(filename, linenum, 'whitespace/declaration', 3,
- 'Declaration has space between type name and %s in %s' %
- (matched.group('pointer_operator'), matched.group(0).strip()))
+ line = clean_lines.elided[linenum]
+ matched = _RE_PATTERN_POINTER_DECLARATION_WHITESPACE.match(line)
+ if matched:
+ error(
+ filename, linenum, 'whitespace/declaration', 3,
+ 'Declaration has space between type name and %s in %s' %
+ (matched.group('pointer_operator'), matched.group(0).strip()))