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.

Stefan

_______________________________________________
Cython-dev mailing list
[email protected]
http://codespeak.net/mailman/listinfo/cython-dev

Reply via email to