On Mar 23, 2009, at 11:23 PM, Stefan Behnel wrote: > > Robert Bradshaw wrote: >> On Mar 18, 2009, at 12:44 PM, Stefan Behnel wrote: >>> Robert Bradshaw wrote: >>>> I started a CEP up at http://wiki.cython.org/enhancements/division >>>> which attempts to summarize the issues brought up on this thread. >>> I added the pros and cons that I could come up with to each >>> solution. Maybe >>> others can add to that. >> >> I think we've waited long enough for everyone to give input. I say we >> change the default to python-style division, implementing a compiler >> directive and cython.cdiv/cmod methods to obtain the old behavior. > > +1 > > >> This is a big enough change it should probably wait 'till 0.12 > > It should still go into cython-unstable as soon as possible so that > users > can test their code with it.
Yep, will do. >> we should maybe start printing out a warning in the next release. > > A warning will be somewhat cumbersome to users. The change won't > affect, > say, 90% of the code using "%", but it will require everyone to > care about > it. I'm not saying we shouldn't do that, just that it's a heavy > thing to do > (that, is: if we don't kill performance entirely and check the > value at > runtime to emit a warning only when it's necessary). I'm thinking a single (compile time) warning if % is used on cdef ints. - Robert _______________________________________________ Cython-dev mailing list [email protected] http://codespeak.net/mailman/listinfo/cython-dev
