Matěj Laitl, 09.11.2010 01:06: > On Mon, Nov 8, 2010 at 11:59, Stefan Behnel wrote: >> Stefan Behnel, 08.11.2010 09:03: >>> Matěj Laitl, 08.11.2010 01:13: >>>> Sadly, I'm clueless how to fix it completely. It would be great if a >>>> fix for this bug could make it into 0.13.1 as my code [1] depends on >>>> it. >>> >>> Your patch is a good hint, I'm looking into it. >> >> What you do in your patch would provide a quick fix. >> >> Personally, I think the right way to really fix this would be to finally >> refactor def functions into cdef function wrappers. That way, overriding >> pure def functions in .pxd files would simply enable the underlying cdef >> function as being callable, instead of needing to build a new one from >> scratch. And we could even override def functions as pure cdef functions by >> simply dropping the wrapper entirely. > > That would be great in long-term view, but I see you've commited more > elaborate fix for the bug - that's equally good for now. Thanks for > that, Stefan! I'll have an opportunity to test your patch on > moderately complicated code tomorrow, I'll report back if something > goes wrong.
You should report back even if it works for you. It's more fun to hear about things that work than about things that don't work. Stefan _______________________________________________ Cython-dev mailing list [email protected] http://codespeak.net/mailman/listinfo/cython-dev
