Yeah I didn't realize there was more to it.  Fixing the segv is a given, 
the other might not be.  The other part doesn't seem overly crazy, but 
good to contact the author regardless.

Hannes Janetzek wrote:
> Am Sun, 06 Jan 2008 18:09:10 -0500
> schrieb Christopher Michael <[EMAIL PROTECTED]>:
>
>   
>> Enlightenment CVS wrote:
>>     
>>> Enlightenment CVS committal
>>>
>>> Author  : jeffdameth
>>> Project : e_modules
>>> Module  : forecasts
>>>
>>> Dir     : e_modules/forecasts
>>>
>>>
>>> Modified Files:
>>>     e_mod_main.c 
>>>
>>>
>>> Log Message:
>>> forcast fix:
>>> - initialize inst->popup with NULL
>>> - only update popup when necessary
>>>
>>>       
>> --Snip--
>>
>> Did you check with the author of the module about these changes? CVS 
>> access is not a license to go willy-nilly on other people's code.
>>
>> dh
>>
>>     
>  
> Sorry, no I didnt. I mentioned the patch on irc and mekius said I 
> could commit this patch if it fixes a segv. but maybe he just meant
> the initialize part. I will ask in future _always_ the author.
>
> regards,
> Hannes  
>
>
>   
>> -------------------------------------------------------------------------
>> This SF.net email is sponsored by: Microsoft
>> Defy all challenges. Microsoft(R) Visual Studio 2005.
>> http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
>> _______________________________________________
>> enlightenment-devel mailing list
>> enlightenment-devel@lists.sourceforge.net
>> https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
>>     
>
> -------------------------------------------------------------------------
> This SF.net email is sponsored by: Microsoft
> Defy all challenges. Microsoft(R) Visual Studio 2005.
> http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
> _______________________________________________
> enlightenment-devel mailing list
> enlightenment-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
>   


-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to