[SciPy-dev] Bugfix for weave's catalog.

[Fernando Perez]

Prabhu Ramachandran wrote:
>>>>>>"FP" == Fernando Perez <Fernando.Perez at colorado.edu> writes:
> 
> 
>     FP> Hi all, there has been a long-standing bug in weave.inline,
>     FP> where it recompiles stuff it shouldn't.  I was finally able to
> [...]
> 
>     FP> I can commit this, but I'd greatly appreciate it if someone
>     FP> else has a look at it first (Pearu, RKern?).  I know that
>     FP> nobody likes to muck with the weave code, as it's a bit
>     FP> complex, but I'd feel safer if at least one more pair of
>     FP> eyeballs has a look at this before I commit.
> 
> Thanks for tracking this down.  I just read the patch and it seems OK,
> however, do you have a test case to go with the fix?  If so it would
> be useful to have the test checked in as well.

Well, the problem is that I was never able to see the problem with little 
weave.inline snippets.  I ended up using some code which relies on a large 
internal library of mine, but which at least would trigger the misbehavior 
every time.  Part of why it took me so long to work on this, is that I had 
never been able to identify a reliable way to reproduce the problem, until 
this code started showing it on every run.

But unfortunately it's a big library for internal use, so it won't work for a 
scipy test.  I'm sure it can be triggered with smaller examples, but all the 
ones I tried failed to show it.  Frankly, I don't quite understand why, 
though.  From the looks of the bug, it should occur every time...

If someone else has a small standalone example which shows the problem 
reliably, that would be great to have.

Cheers,

f