diff -r a14732e08fec -r b56bf98c8afb mercurial/revset.py --- a/mercurial/revset.py Sun Feb 14 19:48:33 2016 +0900 +++ b/mercurial/revset.py Sun Feb 14 20:27:08 2016 +0900 @@ -2267,17 +2267,6 @@ return tree[1] return None -def _checkaliasarg(tree, known=None): - """Check tree contains no _aliasarg construct or only ones which - value is in known. Used to avoid alias placeholders injection. - """ - if isinstance(tree, tuple): - arg = _getaliasarg(tree) - if arg is not None and (not known or arg not in known): - raise error.UnknownIdentifier('_aliasarg', []) - for t in tree: - _checkaliasarg(t, known) - # the set of valid characters for the initial letter of symbols in # alias declarations and definitions _aliassyminitletters = set(c for c in [chr(i) for i in xrange(256)] @@ -2443,8 +2432,6 @@ try: self.replacement = _parsealiasdefn(value, self.args) - # Check for placeholder injection - _checkaliasarg(self.replacement, self.args) except error.ParseError as inst: self.error = _('failed to parse the definition of revset alias' ' "%s": %s') % (self.name, parseerrordetail(inst))