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

Unified Diff: third_party/WebKit/Tools/Scripts/webkitpy/style/main.py

Issue 2188623002: Change logging statements to not use the "%" operator. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 5 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/style/main.py
diff --git a/third_party/WebKit/Tools/Scripts/webkitpy/style/main.py b/third_party/WebKit/Tools/Scripts/webkitpy/style/main.py
index e6c88d8e80c1cc8bbe9125eae7d020c43ad12837..a5fe9bb4f7e28a6aab4e7521942b33d9ba2d8b04 100644
--- a/third_party/WebKit/Tools/Scripts/webkitpy/style/main.py
+++ b/third_party/WebKit/Tools/Scripts/webkitpy/style/main.py
@@ -75,7 +75,7 @@ def change_directory(filesystem, checkout_root, paths):
# checkout root. Interpret all of them relative to the
# current working directory, and do not change the current
# working directory.
- _log.warn(
+ _log.warning(
"""Path-dependent style checks may not work correctly:
One of the given paths is outside the WebKit checkout of the current
@@ -86,8 +86,8 @@ def change_directory(filesystem, checkout_root, paths):
Pass only files below the checkout root to ensure correct results.
See the help documentation for more info.
-"""
- % (path, checkout_root))
+""",
+ path, checkout_root)
return paths
rel_paths.append(rel_path)
@@ -158,6 +158,6 @@ class CheckWebKitStyle(object):
file_count = file_reader.file_count
delete_only_file_count = file_reader.delete_only_file_count
- _log.info("Total errors found: %d in %d files" % (error_count, file_count))
+ _log.info("Total errors found: %d in %d files", error_count, file_count)
# We fail when style errors are found or there are no checked files.
return error_count > 0 or (file_count == 0 and delete_only_file_count == 0)

Powered by Google App Engine
This is Rietveld 408576698