Closed GoogleCodeExporter closed 9 years ago
Thank you Alexandr for the bug report and the patches. Sorry for the delay. I
have
fixed the Unicode issues in diff_fromDelta and patch_fromText. In both cases I
added:
if type(text) == unicode:
text = text.encode("ascii")
These are two functions which are expecting a subset of ASCII characters.
However, your patch also made changes to diff_text1, diff_text2, patch_apply
and patch_obj.__str__. Despite many tests, I am unable to find scenarios where
the existing code fails when passed Unicode. An example testcase would be most
apreciated.
In the mean time, I've pushed out a new version which includes the Unicode
fixes for
diff_fromDelta and patch_fromText in the Python version, as well as a new unit
test
in all three versions which verifies the behaviour of invalid Unicode sequences
(e.g.
%c3%xy).
Original comment by neil.fra...@gmail.com
on 14 May 2008 at 7:47
Original issue reported on code.google.com by
sashul...@gmail.com
on 11 May 2008 at 1:12Attachments: