tools-webrtc/check_package_boundaries.py - Issue 2737233003: Revert of PyLint fixes for tools-webrtc and webrtc/tools

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Unified Diff: tools-webrtc/check_package_boundaries.py

Issue 2737233003: Revert of PyLint fixes for tools-webrtc and webrtc/tools (Closed)

Patch Set: Created 3 years, 9 months ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

Index: tools-webrtc/check_package_boundaries.py

diff --git a/tools-webrtc/check_package_boundaries.py b/tools-webrtc/check_package_boundaries.py

index ec1521b56a9825aa3ff740e80cf15e2672999f06..eb791fffb27e3ea5252c345fe705e19b00c70628 100644

--- a/tools-webrtc/check_package_boundaries.py

+++ b/tools-webrtc/check_package_boundaries.py

@@ -39,7 +39,7 @@

self.log_level = DISPLAY_LEVEL

self.messages_left = messages_left

- def Log(self, build_file_path, line_number, target_name, source_file,

+ def log(self, build_file_path, line_number, target_name, source_file,

subpackage):

if self.messages_left is not None:

if not self.messages_left:

@@ -91,7 +91,7 @@

line_number = subpackages_match.group('line_number')

if subpackage:

found_violations = True

- logger.Log(build_file_path, line_number, target_name, source_file,

+ logger.log(build_file_path, line_number, target_name, source_file,

subpackage)

return found_violations