Message86936
Reviewers: report_bugs.python.org, Benjamin,
Message:
Issues fixed in r72188.
http://codereview.appspot.com/52081/diff/1/5
File Doc/library/codecs.rst (right):
http://codereview.appspot.com/52081/diff/1/5#newcode326
Line 326: In addition, the following error handlers are specific to only
selected
On 2009/05/01 21:25:44, Benjamin wrote:
> "In addition, the following error handlers are specific to a single
codec."
> sounds better
Done.
http://codereview.appspot.com/52081/diff/1/5#newcode335
Line 335:
On 2009/05/01 21:25:44, Benjamin wrote:
> There should probably be a versionchanged directive indicating that
"surrogates"
> was added in 3.1.
Done.
http://codereview.appspot.com/52081/diff/1/6
File Lib/test/test_codecs.py (right):
http://codereview.appspot.com/52081/diff/1/6#newcode544
Line 544: def test_surrogates(self):
On 2009/05/01 21:25:44, Benjamin wrote:
> I think this should be split into 2 tests. "test_lone_surrogates" and
> "test_surrogate_handler".
Done.
http://codereview.appspot.com/52081/diff/1/4
File Objects/unicodeobject.c (right):
http://codereview.appspot.com/52081/diff/1/4#newcode157
Line 157: static PyObject *unicode_encode_call_errorhandler(const char
*errors,
On 2009/05/01 21:25:44, Benjamin wrote:
> These prototypes are longer than 80 chars some places. I don't think
the
> arguments need to line up with the starting parenthesis.
Done.
http://codereview.appspot.com/52081/diff/1/4#newcode2393
Line 2393: s, size, &exc, i-1, i, &newpos);
On 2009/05/01 21:25:44, Benjamin wrote:
> "exc" is never Py_DECREFed.
Done.
http://codereview.appspot.com/52081/diff/1/4#newcode4110
Line 4110: if (!PyUnicode_Check(repunicode)) {
On 2009/05/01 21:25:44, Benjamin wrote:
> Is there a test of this case somewhere?
No. This is temporary - for PEP 383, I will have to support error
handlers returning bytes here, also.
http://codereview.appspot.com/52081/diff/1/2
File Python/codecs.c (right):
http://codereview.appspot.com/52081/diff/1/2#newcode758
Line 758: if (PyObject_IsInstance(exc, PyExc_UnicodeEncodeError)) {
On 2009/05/01 21:25:44, Benjamin wrote:
> I believe PyErr_GivenExceptionMatches is more appropriate here, but
given the
> rest of the file uses PyObject_IsInstance, it likely doesn't matter.
No. The interface is that an exception instance must be passed;
GivenExceptionMatches would also allow for tuples and types.
http://codereview.appspot.com/52081/diff/1/2#newcode771
Line 771: return NULL;
On 2009/05/01 21:25:44, Benjamin wrote:
> This is leaks "object".
Done.
Please review this at http://codereview.appspot.com/52081
Affected files:
M Doc/library/codecs.rst
M Lib/test/test_bytes.py
M Lib/test/test_codecs.py
M Lib/test/test_unicode.py
M Lib/test/test_unicodedata.py
M Objects/unicodeobject.c
M Python/codecs.c
M Python/marshal.c |
|
Date |
User |
Action |
Args |
2009-05-02 09:44:08 | loewis | set | recipients:
+ loewis, lemburg, Rhamphoryncus, pitrou, benjamin.peterson, jwilk, ezio.melotti |
2009-05-02 09:44:06 | loewis | link | issue3672 messages |
2009-05-02 09:44:05 | loewis | create | |
|