diff options
| author | Sen Jiang <senj@google.com> | 2017-08-15 18:20:24 -0700 |
|---|---|---|
| committer | Sen Jiang <senj@google.com> | 2017-08-16 11:18:36 -0700 |
| commit | 1048559a5c8b6c897aee6c8095035fc7724ae340 (patch) | |
| tree | f6ef77bd216c698fdfb2ebc40e33d4f8dd49ea6d /scripts/update_device.py | |
| parent | 5a46a18f4bedb1c5ee4a77050716b6809a71a9e4 (diff) | |
Fix pylint warnings in update_device.py
We weren't actually using our own pylintrc file in repo upload hook,
which will be fixed by https://android-review.googlesource.com/#/c/platform/tools/repohooks/+/458996/
Even though both camel case and snake case are allowed for function
and method names, they still have to be consistent in a single file.
https://pylint.readthedocs.io/en/latest/user_guide/options.html#multiple-naming-styles
Bug: 27316596
Test: repo upload
Change-Id: I98adba957a547115ef3d8b00fe2e3e4931707b31
Diffstat (limited to 'scripts/update_device.py')
| -rwxr-xr-x | scripts/update_device.py | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/scripts/update_device.py b/scripts/update_device.py index 75c58a7b..63d0fc93 100755 --- a/scripts/update_device.py +++ b/scripts/update_device.py @@ -93,7 +93,7 @@ class UpdateHandler(BaseHTTPServer.BaseHTTPRequestHandler): """ @staticmethod - def _ParseRange(range_str, file_size): + def _parse_range(range_str, file_size): """Parse an HTTP range string. Args: @@ -141,11 +141,11 @@ class UpdateHandler(BaseHTTPServer.BaseHTTPRequestHandler): self.send_response(200) stat = os.fstat(f.fileno()) - start_range, end_range = self._ParseRange(self.headers.get('range'), - stat.st_size) + start_range, end_range = self._parse_range(self.headers.get('range'), + stat.st_size) logging.info('Serving request for %s from %s [%d, %d) length: %d', - self.path, self.serving_payload, start_range, end_range, - end_range - start_range) + self.path, self.serving_payload, start_range, end_range, + end_range - start_range) self.send_header('Accept-Ranges', 'bytes') self.send_header('Content-Range', |