Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(9)

Unified Diff: third_party/WebKit/Tools/Scripts/webkitpy/common/checkout/diff_parser.py

Issue 2014063002: Run format-webkit on webkitpy code (without string conversion). (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 7 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:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Tools/Scripts/webkitpy/common/checkout/diff_parser.py
diff --git a/third_party/WebKit/Tools/Scripts/webkitpy/common/checkout/diff_parser.py b/third_party/WebKit/Tools/Scripts/webkitpy/common/checkout/diff_parser.py
index 41bb16de21147d78f1348e5ac27a72f42c3f7b8d..6aaec6f90d0ece893b1c4e90b821335a5b545eaf 100644
--- a/third_party/WebKit/Tools/Scripts/webkitpy/common/checkout/diff_parser.py
+++ b/third_party/WebKit/Tools/Scripts/webkitpy/common/checkout/diff_parser.py
@@ -87,6 +87,7 @@ _PROCESSING_CHUNK = 3
class DiffFile(object):
+
"""Contains the information for one file in a patch.
qyearsley 2016/05/27 16:06:03 Blank line before class docstring is not necessari
Dirk Pranke 2016/06/01 00:06:47 I'm not particularly a fan of the blank line.
qyearsley 2016/06/01 21:24:22 Alright -- changed formatter/main.py to not add th
The field "lines" is a list which contains tuples in this format:
@@ -119,6 +120,7 @@ class DiffFile(object):
# If this is going to be called DiffParser, it should be a re-useable parser.
# Otherwise we should rename it to ParsedDiff or just Diff.
class DiffParser(object):
+
"""A parser for a patch file.
The field "files" is a dict whose key is the filename and value is

Powered by Google App Engine
This is Rietveld 408576698