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

Unified Diff: third_party/WebKit/Tools/Scripts/webkitpy/style/filereader.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/filereader.py
diff --git a/third_party/WebKit/Tools/Scripts/webkitpy/style/filereader.py b/third_party/WebKit/Tools/Scripts/webkitpy/style/filereader.py
index 13d021983f3090f40777d13d4cd8f190658f880b..9226dd81c4d566c227f923c100df4d0a4c28fd27 100644
--- a/third_party/WebKit/Tools/Scripts/webkitpy/style/filereader.py
+++ b/third_party/WebKit/Tools/Scripts/webkitpy/style/filereader.py
@@ -112,19 +112,19 @@ class TextFileReader(object):
self.file_count += 1
if not self.filesystem.exists(file_path) and file_path != "-":
- _log.error("File does not exist: '%s'" % file_path)
+ _log.error("File does not exist: '%s'", file_path)
sys.exit(1) # FIXME: This should throw or return instead of exiting directly.
if not self._processor.should_process(file_path):
- _log.debug("Skipping file: '%s'" % file_path)
+ _log.debug("Skipping file: '%s'", file_path)
return
- _log.debug("Processing file: '%s'" % file_path)
+ _log.debug("Processing file: '%s'", file_path)
try:
lines = self._read_lines(file_path)
except IOError as err:
message = ("Could not read file. Skipping: '%s'\n %s" % (file_path, err))
- _log.warn(message)
+ _log.warning(message)
return
self._processor.process(lines, file_path, **kwargs)

Powered by Google App Engine
This is Rietveld 408576698