[issue16495] bytes_decode() unnecessarily examines encoding

2012-12-07 Thread Chris Jerdonek
Changes by Chris Jerdonek : -- resolution: -> fixed stage: needs patch -> committed/rejected status: open -> closed ___ Python tracker ___ __

[issue16495] bytes_decode() unnecessarily examines encoding

2012-12-07 Thread Roundup Robot
Roundup Robot added the comment: New changeset 3fb9e38b00e8 by Chris Jerdonek in branch 'default': Issue #16495: remove extraneous NULL encoding check from bytes_decode(). http://hg.python.org/cpython/rev/3fb9e38b00e8 -- nosy: +python-dev ___ Python t

[issue16495] bytes_decode() unnecessarily examines encoding

2012-12-07 Thread Andrew Svetlov
Andrew Svetlov added the comment: LGTM -- ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.or

[issue16495] bytes_decode() unnecessarily examines encoding

2012-12-03 Thread Andrew Svetlov
Changes by Andrew Svetlov : -- nosy: +asvetlov ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.pyth

[issue16495] bytes_decode() unnecessarily examines encoding

2012-11-28 Thread Ezio Melotti
Changes by Ezio Melotti : -- nosy: +ezio.melotti ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.py

[issue16495] bytes_decode() unnecessarily examines encoding

2012-11-26 Thread Jesús Cea Avión
Changes by Jesús Cea Avión : -- nosy: +jcea ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.

[issue16495] bytes_decode() unnecessarily examines encoding

2012-11-18 Thread Chris Jerdonek
Chris Jerdonek added the comment: Patch attached. The tests pass. -- keywords: +patch Added file: http://bugs.python.org/file28039/issue-16495-1-default.patch ___ Python tracker ___

[issue16495] bytes_decode() unnecessarily examines encoding

2012-11-17 Thread Chris Jerdonek
Chris Jerdonek added the comment: For the record, there is a test for the default encoding: http://hg.python.org/cpython/file/e9af9b1ca67e/Lib/test/test_bytes.py#l209 -- ___ Python tracker

[issue16495] bytes_decode() unnecessarily examines encoding

2012-11-17 Thread Serhiy Storchaka
Changes by Serhiy Storchaka : -- stage: -> needs patch ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://

[issue16495] bytes_decode() unnecessarily examines encoding

2012-11-17 Thread Chris Jerdonek
New submission from Chris Jerdonek: The code below in bytes_decode() is unnecessary: if (encoding == NULL) encoding = PyUnicode_GetDefaultEncoding(); (from http://hg.python.org/cpython/file/e9af9b1ca67e/Objects/bytesobject.c#l2230 ) because PyUnicode_FromEncodedObject() already ha