----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33304/#review80484 -----------------------------------------------------------
proton-c/bindings/python/proton/reactor.py <https://reviews.apache.org/r/33304/#comment130401> It's not calling next on itself, rather it calls next on the contained iterator (to the self.values array). Agreed that it looks a bit weird, but the indent appears to be a 'circular iterator' - one that never throws StopIteration but returns to the sequence start instead. At least that's how it works when I run it by hand. - Kenneth Giusti On April 17, 2015, 1:17 p.m., Kenneth Giusti wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/33304/ > ----------------------------------------------------------- > > (Updated April 17, 2015, 1:17 p.m.) > > > Review request for qpid, Alan Conway, Gordon Sim, Rafael Schloming, and > Robbie Gemmell. > > > Bugs: PROTON-490 > https://issues.apache.org/jira/browse/PROTON-490 > > > Repository: qpid-proton-git > > > Description > ------- > > First of a series of patches, this patch shows the changes made by the > futurize tool when run on the proton-c/bindings directory. > > These changes preserve python2 compatibility while avoiding language features > deprecated by python3. These changes don't enable python3 support, they > merely eliminate the syntax/features that won't port. > > > Diffs > ----- > > proton-c/bindings/python/proton/__init__.py eeb7768 > proton-c/bindings/python/proton/reactor.py 467bb76 > proton-c/bindings/python/proton/utils.py b5ecdf7 > > Diff: https://reviews.apache.org/r/33304/diff/ > > > Testing > ------- > > Ran the unit tests - all pass including Jython-based tests. > > > Thanks, > > Kenneth Giusti > >
