indygreg added a comment.
In https://phab.mercurial-scm.org/D4465#68517, @pulkit wrote:
> Queued the cbor patches. I think following can improve the code more:
>
> - Having a `cborutil.decodeone()` function which can make sure we only have
one dcoded value and using that fn instead
This revision was automatically updated to reflect the committed changes.
Closed by commit rHGcdb56f295b03: httppeer: use our CBOR decoder (authored by
indygreg, committed by ).
REPOSITORY
rHG Mercurial
CHANGES SINCE LAST UPDATE
https://phab.mercurial-scm.org/D4465?vs=10762=10782
REVISION
pulkit added a comment.
Queued the cbor patches. I think following can improve the code more:
- Having a `cborutil.decodeone()` function which can make sure we only have
one dcoded value and using that fn instead of `decodeall()[0]`
- A `cborutil.encode()` function to use where we
indygreg created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.
REVISION SUMMARY
We just implemented our own CBOR decoder. Let's use it in
httppeer.
REPOSITORY
rHG Mercurial
REVISION DETAIL
https://phab.mercurial-scm.org/D4465
AFFECTED