[issue19676] Add the "namereplace" error handler

2014-11-26 Thread Ned Deily
Ned Deily added the comment: Fixed in ce8a8531d29a -- status: open -> closed ___ Python tracker ___ ___ Python-bugs-list mailing list

[issue19676] Add the "namereplace" error handler

2014-11-26 Thread Ned Deily
Ned Deily added the comment: ../../source/Python/codecs.c:1022:16: error: use of undeclared identifier 'out'; did you mean 'outp'? assert(out == start + ressize); ^~~ outp -- nosy: +ned.deily status: closed -> open ___

[issue19676] Add the "namereplace" error handler

2014-11-26 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: Thank you Berker. -- ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https:

[issue19676] Add the "namereplace" error handler

2014-11-26 Thread Roundup Robot
Roundup Robot added the comment: New changeset 21d1571c0533 by Serhiy Storchaka in branch 'default': Issue #19676: Fixed integer overflow issue in "namereplace" error handler. https://hg.python.org/cpython/rev/21d1571c0533 -- ___ Python tracker

[issue19676] Add the "namereplace" error handler

2014-11-25 Thread Roundup Robot
Roundup Robot added the comment: New changeset b6fab008d63a by Berker Peksag in branch 'default': Issue #19676: Tweak documentation a bit. https://hg.python.org/cpython/rev/b6fab008d63a -- ___ Python tracker __

[issue19676] Add the "namereplace" error handler

2014-11-25 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: Thank you all for reviews. > One nit: the name buffer length should be NAME_MAXLEN instead of 100. NAME_MAXLEN is private name available only in Modules/unicodedata.c. Making it public name would be other issue. I have increased buffer size to 256.

[issue19676] Add the "namereplace" error handler

2014-11-25 Thread Roundup Robot
Roundup Robot added the comment: New changeset 32d08aacffe0 by Serhiy Storchaka in branch 'default': Issue #19676: Added the "namereplace" error handler. https://hg.python.org/cpython/rev/32d08aacffe0 -- nosy: +python-dev ___ Python tracker

[issue19676] Add the "namereplace" error handler

2014-11-25 Thread Nick Coghlan
Nick Coghlan added the comment: Patch looks good to me, too. As far as Amaury's question goes, isn't the general reverse operation the same as for the existing backslashreplace handler? That is, decode with the appropriate ASCII compatible encoding (since ASCII compatibility is needed for the

[issue19676] Add the "namereplace" error handler

2014-11-25 Thread Marc-Andre Lemburg
Marc-Andre Lemburg added the comment: The patch looks good. One nit: the name buffer length should be NAME_MAXLEN instead of 100. -- ___ Python tracker ___ _

[issue19676] Add the "namereplace" error handler

2014-11-25 Thread Amaury Forgeot d'Arc
Amaury Forgeot d'Arc added the comment: The patch looks good to me. But it seems that the reverse operation is not possible in the general case: .decode('unicode_escape') assumes a latin-1 or ascii encoding. Should we document this? -- nosy: +amaury.forgeotdarc

[issue19676] Add the "namereplace" error handler

2014-11-25 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: Ping. -- assignee: -> serhiy.storchaka keywords: +needs review versions: +Python 3.5 -Python 3.4 ___ Python tracker ___ _

[issue19676] Add the "namereplace" error handler

2013-11-23 Thread Serhiy Storchaka
Changes by Serhiy Storchaka : -- nosy: +ethan.furman, lemburg, ncoghlan, stevenjd ___ Python tracker ___ ___ Python-bugs-list mailing

[issue19676] Add the "namereplace" error handler

2013-11-20 Thread STINNER Victor
STINNER Victor added the comment: See also issue #18234. -- ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: htt

[issue19676] Add the "namereplace" error handler

2013-11-20 Thread Serhiy Storchaka
New submission from Serhiy Storchaka: The proposed patch adds the "namereplace" error handler. This error handler is almost same as the "backslashreplace" error handler, but use \N{...} escape sequences if there is a character name in Unicode database. Result is a little more human-readable (b