# HG changeset patch # User Pierre-Yves David # Date 1431970035 18000 # Node ID d647f97f88ddb072b48a09186fb2a1d239aa4390 # Parent c69f4f7fe01ae75add6da2be717fd046d2ac63b0 parsers: use 'next' instead of try/except This get rid of another StopIteration abomination. The change in self.current value is supposed to not matter as nobody should be calling '_advance' after that (as per Matt wisdom). diff -r c69f4f7fe01a -r d647f97f88dd mercurial/parser.py --- a/mercurial/parser.py Mon May 18 12:22:44 2015 -0500 +++ b/mercurial/parser.py Mon May 18 12:27:15 2015 -0500 @@ -27,10 +27,7 @@ def _advance(self): 'advance the tokenizer' t = self.current - try: - self.current = self._iter.next() - except StopIteration: - pass + self.current = next(self._iter, None) return t def _match(self, m, pos): 'make sure the tokenizer matches an end condition'