Lisandro Dalcin wrote: > Likely Dag should review this... If you agree on the patch, please > push ... IMHO, this fix should go to 0.11 > I don't think I have anything to do with that code (though there's likely other instances of the same problem in my Buffer.py).
a) Saying that this should go into 0.11 means saying that this specific bug is more important than everything else that sits in milestone 0.11.1 in Trac. We probably put out 0.10 with the same bug and the Cython project survived! We need to have some disipline and get things released -- release early, release often. b) I don't know if I'm the right person to review, I don't know too much about the Python API or size_t in this respect; it looks OK to me. (If I made the API in the first place I'd make the macro contain the entire string, i.e. "d" and "ld", and not simply the "l" or nothing. Some C versions have a "z" for size_t IIRC?) Dag Sverre _______________________________________________ Cython-dev mailing list [email protected] http://codespeak.net/mailman/listinfo/cython-dev
