----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33304/#review80464 -----------------------------------------------------------
proton-c/bindings/python/proton/reactor.py <https://reviews.apache.org/r/33304/#comment130374> This next stuff looks suspicious to me, porticularly if this is indeed a mechanical change resulting from the 2to3 tool. Is this really supposed to recursively call itself in this way? - Rafael Schloming 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 > >
