Message218292
Here are updated patches for 2.7 and HEAD.
I've updated the patches for both python 2 and python 3 and addressed some of the issues here:
1. Added _ctypes_alloc_format_string_with_shape() to ctypes.h
2. Changed the buffer size calculation to 32 *ndim + 3 (for the question asked before, the 32 size is just because this was the size used before the patch....)
3. Changed the sprintf to use the macro: sprintf(buf, "%"PY_FORMAT_SIZE_T"d,", shape[k]);
I really hope this patch makes it to 2.7.7 and python 3 versions, as currently the situation is broken and code that once worked with older numpy versions does not work today (and we have legacy code that stopped working). |
|
Date |
User |
Action |
Args |
2014-05-11 19:57:34 | mattip | set | recipients:
+ mattip, theller, ronaldoussoren, mark.dickinson, pitrou, vstinner, pv, skrah, Tilka |
2014-05-11 19:57:34 | mattip | set | messageid: <1399838254.84.0.374096080226.issue10744@psf.upfronthosting.co.za> |
2014-05-11 19:57:34 | mattip | link | issue10744 messages |
2014-05-11 19:57:34 | mattip | create | |
|