Merge lp:~gz/brz/bencode_recursion into lp:brz
Proposed by
Martin Packman
Status: | Merged |
---|---|
Approved by: | Martin Packman |
Approved revision: | no longer in the source branch. |
Merge reported by: | The Breezy Bot |
Merged at revision: | not available |
Proposed branch: | lp:~gz/brz/bencode_recursion |
Merge into: | lp:brz |
Diff against target: |
230 lines (+50/-37) 4 files modified
breezy/_bencode_pyx.pyx (+20/-24) breezy/python-compat.h (+2/-0) breezy/tests/test__bencode.py (+26/-13) python3.passing (+2/-0) |
To merge this branch: | bzr merge lp:~gz/brz/bencode_recursion |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Jelmer Vernooij | Approve | ||
Review via email: mp+353451@code.launchpad.net |
Commit message
Fix recursion handling in bencode extension
Description of the change
This conflicts with part of the fix in the proposed fix-c-extensions branch:
https:/
I think the right change is a mix of the two? Should bare raise but with the (uncovered?) encode site also updated.
To post a comment you must log in.
This seems simpler than my solution and also fix other stuff (e.g. the warnings).