<div dir="ltr"><br><br><div class="gmail_quote">On Mon, Mar 9, 2015 at 4:16 PM Matt Mackall <<a href="mailto:mpm@selenic.com">mpm@selenic.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Sun, 2015-03-08 at 13:08 -0400, Brett Cannon wrote:<br>
> # HG changeset patch<br>
> # User Brett Cannon <<a href="mailto:brett@python.org" target="_blank">brett@python.org</a>><br>
> # Date 1425834517 14400<br>
> #      Sun Mar 08 13:08:37 2015 -0400<br>
> # Node ID 06babaeea98aaa7c58502bf3d9879f<u></u>ac72fd8d46<br>
> # Parent  ea80bd2775f6612399eb3aedf2444f<u></u>cb11b0cf8a<br>
> hglib: Wrap all application string literals in util.b() (issue4520)<br>
<br>
This is queued for hglib, thanks. I made a few tweaks:<br>
<br>
- summary caps<br>
- drop stray whitespace changes<br>
- fix some overlong lines<br>
<br>
I can't tell you how much str(someinteger).encode('<u></u>latin1') pains me<br>
though. I think we're going to need another helper for that.<br>
<br>
Also, I think we'll want to do "b = str" on 2.x for performance.<br></blockquote><div><br></div><div>SGTM. I'll work those changes into a patch at some point (probably easier for me to get all the changes I have queued in my local copy in first and then do a cleanup pass after).</div></div></div>