Message226915
On 15.09.2014 15:19, STINNER Victor wrote:
>
> STINNER Victor added the comment:
>
> IMO test_codecs_fix1.patch is still needed.
>
> Review of Nick's change:
>
> + interp = PyThreadState_GET()->interp;
> + if (interp->codec_search_path == NULL) {
> + return -1;
> + }
>
> I don't think that this line raises an exception.
Agreed.
Please also add some documentation to the codecs.h file and
perhaps a test for the function itself ;-) |
|
| Date |
User |
Action |
Args |
| 2014-09-15 13:29:44 | lemburg | set | recipients:
+ lemburg, brett.cannon, ncoghlan, pitrou, vstinner, python-dev, zach.ware, serhiy.storchaka |
| 2014-09-15 13:29:44 | lemburg | link | issue22166 messages |
| 2014-09-15 13:29:44 | lemburg | create | |
|