Bug in optimization of range

[dagss]

This:

   cdef int i, n
    n = 10
    for i in range(n):
        if i == 5: n *= 2
    print i

prints out 20.

[dagss]

It should print 9, I think

[dagss]

Note: This must be changed in the for in range-optimization, because "for 0 <= i < n" should have this semantics!

[robertwb]

Works as expected. Committed  http://hg.cython.org/cython-devel/rev/620326b14c3f

This is another reason to keep the for...from syntax though.

[robertwb]

This is a dupe of #196, the above commit was for #208.

[dagss]

No, this is a different bug. Reopening.

Also note that this ticket is assigned to Prajwal Suhas P; see  http://thread.gmane.org/gmane.comp.python.cython.devel/4775

Magnus' patch was only for incrementing-one-to-far; this is about evaluating the endpoints to range once per loop.

Also consider this test case:

"""
 >>> f()
g called
0
1
2
2
"""

cdef int g():
     print "g called"
     return 3

def f():
     cdef int i
     for i in range(g()):
         print i
     print i

[dagss]

That is; I get:

Failed example:
    f()
Expected:
    g called
    0
    1
    2
    2
Got:
    g called
    0
    g called
    1
    g called
    2
    g called
    2

[robertwb]

Finally resolved by #243