On Wed, Oct 15, 2008 at 5:05 PM, Dag Sverre Seljebotn <[EMAIL PROTECTED]> wrote: > Lisandro Dalcin wrote: >> >> This look nicer. However, there is still something that does not work. >> Try to compile the following: >> >> cdef object foo1 >> cdef object foo2 = [] >> >> and then you will see that the declaration of the (unused) 'foo1' is >> not ever emited in the module declaration part. Some optimization >> should be playing games. Or perhaps 'foo1' should be completely >> ignored and never declared/initialized/cleaned-up? >> > > I believe this is an unrelated and long-standing bug? > Anyway, the following patch should fix it (unless we want the opposite > behaviour).
OK, I'll give a try, but perhaps tomorrow. > > Will you commit it together with your testcase? (I.e. try to print "foo1" > from a doctest, which is only in a string and so will trigger this case, and > it should be "None"...) Well, I'll try to make a good test case for all this. But the 'print foo1' will not work, as 'foo1' is not exported in the module dict, as it is cdef stuff (print foo2 in a doctest docstring will not work either). > -- > Dag Sverre > > _______________________________________________ > Cython-dev mailing list > [email protected] > http://codespeak.net/mailman/listinfo/cython-dev > > -- Lisandro Dalcín --------------- Centro Internacional de Métodos Computacionales en Ingeniería (CIMEC) Instituto de Desarrollo Tecnológico para la Industria Química (INTEC) Consejo Nacional de Investigaciones Científicas y Técnicas (CONICET) PTLC - Güemes 3450, (3000) Santa Fe, Argentina Tel/Fax: +54-(0)342-451.1594 _______________________________________________ Cython-dev mailing list [email protected] http://codespeak.net/mailman/listinfo/cython-dev
