On Tue, Feb 14, 2012 at 4:43 PM, Iván Briano (Sachiel)
<[email protected]> wrote:
> 2012/2/14 Enlightenment SVN <[email protected]>:
>> Log:
>> hey cedric.. let's not segv shall we?
>>
>> Author:       raster
>> Date:         2012-02-14 07:03:12 -0800 (Tue, 14 Feb 2012)
>> New Revision: 67938
>> Trac:         http://trac.enlightenment.org/e/changeset/67938
>>
>> Modified:
>>  trunk/edje/src/lib/edje_program.c
>>
>> Modified: trunk/edje/src/lib/edje_program.c
>> ===================================================================
>> --- trunk/edje/src/lib/edje_program.c   2012-02-14 15:02:57 UTC (rev 67937)
>> +++ trunk/edje/src/lib/edje_program.c   2012-02-14 15:03:12 UTC (rev 67938)
>> @@ -873,9 +873,11 @@
>>  {
>>    Edje_Message_Signal emsg;
>>    const char *sep;
>> -   Eina_Bool broadcast = ed->collection->broadcast_signal;
>> +   Eina_Bool broadcast;
>>
>> +   if (!ed->collection) return;
>
> A quick look shows that the code was working without a collection before.
> We are not missing any case by just returning now, are we?

Sending signal to an empty object will not trigger any action.
-- 
Cedric BAIL

------------------------------------------------------------------------------
Keep Your Developer Skills Current with LearnDevNow!
The most comprehensive online learning library for Microsoft developers
is just $99.99! Visual Studio, SharePoint, SQL - plus HTML5, CSS3, MVC3,
Metro Style Apps, more. Free future releases when you subscribe now!
http://p.sf.net/sfu/learndevnow-d2d
_______________________________________________
enlightenment-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to