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

Unified Diff: watchlists.py

Issue 4360002: Largely reduce the number of pylint warnings and fix one typo. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/depot_tools
Patch Set: address comments Created 10 years, 1 month 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
« no previous file with comments | « trychange.py ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: watchlists.py
diff --git a/watchlists.py b/watchlists.py
index 9d38faf49f8fc8cd1979a05e7a0c487b425fb217..57b810cb179f0c3ecc02392032848b4e5ed8f795 100755
--- a/watchlists.py
+++ b/watchlists.py
@@ -106,9 +106,11 @@ class Watchlists(object):
for path in paths:
path = path.replace(os.sep, '/')
for name, rule in self._defns.iteritems():
- if name not in self._watchlists: continue
+ if name not in self._watchlists:
+ continue
rex_str = rule.get('filepath')
- if not rex_str: continue
+ if not rex_str:
+ continue
if re.search(rex_str, path):
map(watchers.add, self._watchlists[name])
return list(watchers)
« no previous file with comments | « trychange.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698