D4465: httppeer: use our CBOR decoder

2018-09-05 Thread indygreg (Gregory Szorc)
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

D4465: httppeer: use our CBOR decoder

2018-09-05 Thread indygreg (Gregory Szorc)
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

D4465: httppeer: use our CBOR decoder

2018-09-05 Thread pulkit (Pulkit Goyal)
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

D4465: httppeer: use our CBOR decoder

2018-09-04 Thread indygreg (Gregory Szorc)
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