On Jan 10, 2009, at 12:08 AM, Stefan Behnel wrote: > Hi, > > Hoyt Koepke wrote: >> beginswith() in ExprNode.py should be startswith() > > Thanks for catching that, I applied the patch. I would expect the > same bug > in Pyrex, BTW. > > It looks like we lack a test case for this specific part of the > code. Could > you come up with something here? > > http://wiki.cython.org/ > HackerGuide#head-6afd0aa52fa6a1de32f6d554b5d30556699e4f18 > > Might fit in here: > > tests/run/consts.pyx > tests/run/ct_DEF.pyx > > >> P.S. Is this the best way to send little updates like these? >> Should I >> send them to the whole list? > > I think it's perfect for small to medium sized patches. More > complex stuff > should go to the bug tracker, but for patches like this, a quick > review > while reading your mail and a quick import is the most efficient > way to > apply them.
I agree. I try and make track tickets for history and completeness (e.g. it makes release notes much easier) but for something like this it is (currently) the best way to go about it. If no one responds, make sure a ticket gets filed so it doesn't get lost in the archives. - Robert _______________________________________________ Cython-dev mailing list [email protected] http://codespeak.net/mailman/listinfo/cython-dev
