Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: sitescripts/subscriptions/knownIssuesParser.py

Issue 29670611: Issue 5844 - Remove redundant parentheses in sitescripts and abpsembly (Closed) Base URL: https://hg.adblockplus.org/sitescripts/
Patch Set: Fixed one more case of redundant parentheses Created Jan. 25, 2018, 7:03 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « sitescripts/subscriptions/bin/updateStatusPage.py ('k') | sitescripts/subscriptions/subscriptionParser.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sitescripts/subscriptions/knownIssuesParser.py
===================================================================
--- a/sitescripts/subscriptions/knownIssuesParser.py
+++ b/sitescripts/subscriptions/knownIssuesParser.py
@@ -186,7 +186,7 @@
if line.find('=') < 0:
print >>sys.stderr, 'Unrecognized line %s' % line
continue
- (key, value) = line.split('=', 1)
+ key, value = line.split('=', 1)
key = key.rstrip()
value = value.lstrip()
if key == 'url':
@@ -199,7 +199,7 @@
def findMatches(it, lang):
global supportedKeys
- (rules, rulesets) = getRules()
+ rules, rulesets = getRules()
for line in it:
match = re.search(r'<([\w\-]+)\s*(.*?)\s*/?>([^<>]*)', line)
@@ -216,9 +216,9 @@
for key, t in supportedKeys.iteritems():
if len(t) == 3:
- (requiredTag, requiredAttrs, requiredValue) = t
+ requiredTag, requiredAttrs, requiredValue = t
else:
- (requiredTag, requiredAttrs) = t
+ requiredTag, requiredAttrs = t
requiredValue = None
requiredAttrs = requiredAttrs.split(' ')
if requiredTag != tag:
« no previous file with comments | « sitescripts/subscriptions/bin/updateStatusPage.py ('k') | sitescripts/subscriptions/subscriptionParser.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld