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

Reply via email to