On Tue, Nov 10, 2009 at 5:46 PM, Enlightenment SVN <no-re...@enlightenment.org> wrote: > Log: > dont segv > Author: sachiel > Date: 2009-11-10 11:46:21 -0800 (Tue, 10 Nov 2009) > New Revision: 43606 > > Modified: > trunk/BINDINGS/python/python-elementary/elementary/elementary.c_elementary_hoversel.pxi > > Modified: > trunk/BINDINGS/python/python-elementary/elementary/elementary.c_elementary_hoversel.pxi > =================================================================== > --- > trunk/BINDINGS/python/python-elementary/elementary/elementary.c_elementary_hoversel.pxi > 2009-11-10 17:03:31 UTC (rev 43605) > +++ > trunk/BINDINGS/python/python-elementary/elementary/elementary.c_elementary_hoversel.pxi > 2009-11-10 19:46:21 UTC (rev 43606) > @@ -110,7 +110,11 @@ > elm_hoversel_label_set(self.obj, label) > > def label_get(self): > - return elm_hoversel_label_get(self.obj) > + cdef char *l > + l = elm_hoversel_label_get(self.obj) > + if not l:
I guess when it's cython, it's better to be explicit and compare to NULL, so: if l == NULL: this way we're safe to it trying any tricks to convert this to python in future (they're all about changing these things and screwing users now and then). -- Gustavo Sverzut Barbieri http://profusion.mobi embedded systems -------------------------------------- MSN: barbi...@gmail.com Skype: gsbarbieri Mobile: +55 (19) 9225-2202 ------------------------------------------------------------------------------ Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day trial. Simplify your report design, integration and deployment - and focus on what you do best, core application coding. Discover what's new with Crystal Reports now. http://p.sf.net/sfu/bobj-july _______________________________________________ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel