Issue1502

The 'string-escape' codec (and its unicode cousin) do not escape the 
escape character, thus fail to roundtrip.

Jython 2.5.1 (Release_2_5_1:6813, Sep 26 2009, 13:47:54)
[Java HotSpot(TM) Client VM (Sun Microsystems Inc.)] on java1.6.0_16
Type "help", "copyright", "credits" or "license" for more information.
>>> "\\x00".encode("string-escape")
'\\x00'
>>> _.decode("string-escape")
'\x00'

One of the issues reported on jython-dev last week pertaining to 
running bazaar on jython.

fixed in r7001

IIRC I tried your patch a long time ago and it broke other things. I opted to go with CPython's slightly less efficient solution. Thanks

Might have been nice to mention what it broke at the time, I could perhaps have posted a follow up.

I have trouble believing that the current behaviour of not escaping the escape character is ever right, even if some other callers might need to be adjusted.

Your commit fails to fix _codecs.unicode_escape_encode as it only touches _codecs.escape_encode - if you want a change in that module rather than PyString they both need doing.

My mistake, your original patch was fine. I must have been thinking of something else. Applied in r7002 with an additional test for unicode-escape.

Thanks for the patch and the followup

Ah, great, thanks for double checking that.