Message191335
> I've attached a new version of the patch that does this check, and also adds a test.
You are right. Your patch LGTM.
> (OTOH, looking at the ncurses 5.9 source code, set_panel_userptr() only returns an error if the panel object is NULL, which should never happen because Python reports an error if the panel creation fails. So maybe the rc == ERR check is pointless.)
Future versions or alternative implementations can returns an error in other circumstances. |
|
Date |
User |
Action |
Args |
2013-06-17 14:06:28 | serhiy.storchaka | set | recipients:
+ serhiy.storchaka, akuchling, ishimoto, ezio.melotti, python-dev |
2013-06-17 14:06:28 | serhiy.storchaka | set | messageid: <1371477988.0.0.0181779517201.issue18113@psf.upfronthosting.co.za> |
2013-06-17 14:06:27 | serhiy.storchaka | link | issue18113 messages |
2013-06-17 14:06:27 | serhiy.storchaka | create | |
|