[PATCH 5 of 6] parser: remove unused binding parameter from suffix action
Yuya Nishihara
yuya at tcha.org
Fri Jul 17 07:34:06 CDT 2015
# HG changeset patch
# User Yuya Nishihara <yuya at tcha.org>
# Date 1436098279 -32400
# Sun Jul 05 21:11:19 2015 +0900
# Node ID 8836e46ccea8e45da42b0e2eda90cb0e88880353
# Parent 3cdc85527cbbe58d148e3c2d9bdeb3281acb1976
parser: remove unused binding parameter from suffix action
Because a suffix action never involves subsequent tokens, it should have
no binding strength nor closing character.
diff --git a/mercurial/parser.py b/mercurial/parser.py
--- a/mercurial/parser.py
+++ b/mercurial/parser.py
@@ -63,7 +63,7 @@ class parser(object):
# handle infix rules, take as suffix if unambiguous
infix, suffix = self._elements[token][3:]
if suffix and not (infix and self._hasnewterm()):
- expr = (suffix[0], expr)
+ expr = (suffix, expr)
elif infix:
expr = (infix[0], expr, self._parseoperand(*infix[1:]))
else:
diff --git a/mercurial/revset.py b/mercurial/revset.py
--- a/mercurial/revset.py
+++ b/mercurial/revset.py
@@ -119,18 +119,16 @@ elements = {
"(": (21, None, ("group", 1, ")"), ("func", 1, ")"), None),
"##": (20, None, None, ("_concat", 20), None),
"~": (18, None, None, ("ancestor", 18), None),
- "^": (18, None, None, ("parent", 18), ("parentpost", 18)),
+ "^": (18, None, None, ("parent", 18), "parentpost"),
"-": (5, None, ("negate", 19), ("minus", 5), None),
- "::": (17, None, ("dagrangepre", 17), ("dagrange", 17),
- ("dagrangepost", 17)),
- "..": (17, None, ("dagrangepre", 17), ("dagrange", 17),
- ("dagrangepost", 17)),
- ":": (15, None, ("rangepre", 15), ("range", 15), ("rangepost", 15)),
+ "::": (17, None, ("dagrangepre", 17), ("dagrange", 17), "dagrangepost"),
+ "..": (17, None, ("dagrangepre", 17), ("dagrange", 17), "dagrangepost"),
+ ":": (15, None, ("rangepre", 15), ("range", 15), "rangepost"),
"not": (10, None, ("not", 10), None, None),
"!": (10, None, ("not", 10), None, None),
"and": (5, None, None, ("and", 5), None),
"&": (5, None, None, ("and", 5), None),
- "%": (5, None, None, ("only", 5), ("onlypost", 5)),
+ "%": (5, None, None, ("only", 5), "onlypost"),
"or": (4, None, None, ("or", 4), None),
"|": (4, None, None, ("or", 4), None),
"+": (4, None, None, ("or", 4), None),
More information about the Mercurial-devel
mailing list