D6044: phabricator: convert conduit response JSON unicode to bytes inside callconduit
Kwan (Ian Moody)
phabricator at mercurial-scm.org
Mon Mar 4 12:12:40 EST 2019
Kwan added a comment.
In https://phab.mercurial-scm.org/D6044#88262, @yuja wrote:
> > - parsed = json.loads(body) + parsed = pycompat.rapply(lambda x: encoding.unitolocal(x) + if isinstance(x, unicode) else x, json.loads(body))
>
> s/unicode/pycompat.unicode/
Thanks, done.
> Perhaps some of `r''` would have to be changed to `b''` since dict keys
> are now byte strings. See the wiki page for our Py3 hacks.
>
> https://www.mercurial-scm.org/wiki/Python3
Ah, good point, thanks. Would it be worth keeping the keys as r'' strings? rapply can fairly easily be extended with an optional notkeys boolean that would allow doing so.
REPOSITORY
rHG Mercurial
REVISION DETAIL
https://phab.mercurial-scm.org/D6044
To: Kwan, #hg-reviewers
Cc: yuja, mercurial-devel
More information about the Mercurial-devel
mailing list