[Cython] Fix for #196 (for loop bug)

[Magnus Lie Hetland]

On Feb 8, 2009, at 12:45, Stefan Behnel wrote:

> Hi,
>
> Magnus Lie Hetland wrote:
>> Should I upload a patch to the tracker?
>
> Sure, that will allow us to take a look at it and compare it to the
> while-loop alternative. Just use "hg export" to keep the meta data.

Done:

http://trac.cython.org/cython_trac/attachment/ticket/208/if_patch.txt

> That's my impression, too. I attached an old (incomplete) patch I  
> had lying
> around, don't know if it's still any good.

Cool. Don't have time to look at it right now, myself, though.

>> there also seemed to be a problem when running
>> with the refnanny.
>
> Could you be a bit more specific here? :)

My mistake. It seems it was rather some cpp stuff that gave me some  
lost refcount messages. Ran it with --no-cpp and it went away.

>> the tests work without the refnanny (except for
>> the NumPy stuff, which all fails because I haven't properly installed
>> NumPy at the moment;
>
> It should actually check for numpy being installed and just skip the  
> test
> if not. At least, that's what it does here.

Hm. It seems I actually do have numpy installed for the same Python  
that I used for this. But the error I get is:

numpy_test.c:129:31: error: numpy/arrayobject.h: No such file or  
directorynumpy_test.c:129:31:
  error: numpy/arrayobject.h: No such file or directory

Followed by lots of "warning: data definition has no type or storage  
class" and syntax errors and undeclared objects in numpy_test.c.

The errors I end up with when running with no switches:

ERROR: compiling (c) and running numpy_test
ERROR: compiling (cpp) and running numpy_test
ERROR: compiling (cpp) and running withstat
FAIL: Doctest: tryfinally
     Error: References leaked:
       Acquired on lines: 396

(The last one appears twice.)

-- 
Magnus Lie Hetland
http://hetland.org