Re: [E-devel] E SVN: andreas trunk/e/src/modules/conf_screensaver

2009-04-14 Thread Michael Jennings
On Tuesday, 14 April 2009, at 20:01:10 (-0300), Gustavo Sverzut Barbieri wrote: > lol... the *warning* is the good thing about not initializing > variables: you know when you miss it. As I said, if it was useful to > init things to 0, then gcc would do it already, don't you think? If it weren't u

Re: [E-devel] E SVN: andreas trunk/e/src/modules/conf_screensaver

2009-04-14 Thread Gustavo Sverzut Barbieri
On Tue, Apr 14, 2009 at 8:19 PM, Michael Jennings wrote: > On Tuesday, 14 April 2009, at 18:19:57 (-0300), > Gustavo Sverzut Barbieri wrote: > >>  yes, it is better, because you can know when you're not using them >> and if you are using them without proper initialization, so you would >> not hide

Re: [E-devel] E SVN: andreas trunk/e/src/modules/conf_screensaver

2009-04-14 Thread Michael Jennings
On Tuesday, 14 April 2009, at 18:19:57 (-0300), Gustavo Sverzut Barbieri wrote: > yes, it is better, because you can know when you're not using them > and if you are using them without proper initialization, so you would > not hide a bug, possibly a segfault. See: > >char *a; >... >

Re: [E-devel] E SVN: andreas trunk/e/src/modules/conf_screensaver

2009-04-14 Thread Gustavo Sverzut Barbieri
On Tue, Apr 14, 2009 at 7:50 PM, Michael Jennings wrote: > On Tuesday, 14 April 2009, at 16:53:44 (-0300), > Gustavo Sverzut Barbieri wrote: > >> But initializing pointers to NULL or variables to 0 is not good, if >> it was be sure that compilers would do that automatically. It's >> easier to hide

Re: [E-devel] E SVN: andreas trunk/e/src/modules/conf_screensaver

2009-04-14 Thread Michael Jennings
On Tuesday, 14 April 2009, at 18:09:50 (-0300), Gustavo Sverzut Barbieri wrote: > Ok, my last email, do what you wish as this is not critical, but > it's not *safer*. This is a false claim. it's bad programing habit > and will make debug with valgrind harder. Just like zeroing pointers > and memor

Re: [E-devel] E SVN: andreas trunk/e/src/modules/conf_screensaver

2009-04-14 Thread Christopher Michael
Michael Jennings wrote: > On Tuesday, 14 April 2009, at 16:53:44 (-0300), > Gustavo Sverzut Barbieri wrote: > >> But initializing pointers to NULL or variables to 0 is not good, if >> it was be sure that compilers would do that automatically. It's >> easier to hide bugs with that, you'll make it h

Re: [E-devel] E SVN: andreas trunk/e/src/modules/conf_screensaver

2009-04-14 Thread Michael Jennings
On Tuesday, 14 April 2009, at 16:53:44 (-0300), Gustavo Sverzut Barbieri wrote: > But initializing pointers to NULL or variables to 0 is not good, if > it was be sure that compilers would do that automatically. It's > easier to hide bugs with that, you'll make it harder to valgrind to > help you :

Re: [E-devel] E SVN: andreas trunk/e/src/modules/conf_screensaver

2009-04-14 Thread Gustavo Sverzut Barbieri
On Tue, Apr 14, 2009 at 6:16 PM, Christopher Michael wrote: > Gustavo Sverzut Barbieri wrote: >> >> On Tue, Apr 14, 2009 at 6:05 PM, Christopher Michael >> wrote: >>> >>> Gustavo Sverzut Barbieri wrote: On Tue, Apr 14, 2009 at 5:16 PM, Christopher Michael wrote: > > Gustav

Re: [E-devel] E SVN: andreas trunk/e/src/modules/conf_screensaver

2009-04-14 Thread Christopher Michael
Gustavo Sverzut Barbieri wrote: > On Tue, Apr 14, 2009 at 6:05 PM, Christopher Michael > wrote: >> Gustavo Sverzut Barbieri wrote: >>> On Tue, Apr 14, 2009 at 5:16 PM, Christopher Michael >>> wrote: Gustavo Sverzut Barbieri wrote: > On Tue, Apr 14, 2009 at 4:09 PM, Andreas Volz > wr

Re: [E-devel] E SVN: andreas trunk/e/src/modules/conf_screensaver

2009-04-14 Thread Gustavo Sverzut Barbieri
On Tue, Apr 14, 2009 at 6:05 PM, Christopher Michael wrote: > Gustavo Sverzut Barbieri wrote: >> >> On Tue, Apr 14, 2009 at 5:16 PM, Christopher Michael >> wrote: >>> >>> Gustavo Sverzut Barbieri wrote: On Tue, Apr 14, 2009 at 4:09 PM, Andreas Volz wrote: > > Am Fri, 10 Ap

Re: [E-devel] E SVN: andreas trunk/e/src/modules/conf_screensaver

2009-04-14 Thread Christopher Michael
Gustavo Sverzut Barbieri wrote: > On Tue, Apr 14, 2009 at 5:16 PM, Christopher Michael > wrote: >> Gustavo Sverzut Barbieri wrote: >>> On Tue, Apr 14, 2009 at 4:09 PM, Andreas Volz wrote: Am Fri, 10 Apr 2009 22:54:45 -0300 schrieb Gustavo Sverzut Barbieri: > On Fri, Apr 10, 2009 at

Re: [E-devel] E SVN: andreas trunk/e/src/modules/conf_screensaver

2009-04-14 Thread Gustavo Sverzut Barbieri
On Tue, Apr 14, 2009 at 5:16 PM, Christopher Michael wrote: > Gustavo Sverzut Barbieri wrote: >> >> On Tue, Apr 14, 2009 at 4:09 PM, Andreas Volz wrote: >>> >>> Am Fri, 10 Apr 2009 22:54:45 -0300 schrieb Gustavo Sverzut Barbieri: >>> On Fri, Apr 10, 2009 at 7:34 PM, Enlightenment SVN w

Re: [E-devel] E SVN: andreas trunk/e/src/modules/conf_screensaver

2009-04-14 Thread Christopher Michael
Gustavo Sverzut Barbieri wrote: > On Tue, Apr 14, 2009 at 4:09 PM, Andreas Volz wrote: >> Am Fri, 10 Apr 2009 22:54:45 -0300 schrieb Gustavo Sverzut Barbieri: >> >>> On Fri, Apr 10, 2009 at 7:34 PM, Enlightenment SVN >>> wrote: >>> - now used Eina_List for storage (I hope I used it correct.

Re: [E-devel] E SVN: andreas trunk/e/src/modules/conf_screensaver

2009-04-14 Thread Gustavo Sverzut Barbieri
On Tue, Apr 14, 2009 at 4:09 PM, Andreas Volz wrote: > Am Fri, 10 Apr 2009 22:54:45 -0300 schrieb Gustavo Sverzut Barbieri: > >> On Fri, Apr 10, 2009 at 7:34 PM, Enlightenment SVN >> wrote: >> >> >  - now used Eina_List for storage (I hope I used it correct...) >> >> > +   Eina_List *l = NULL; >>

Re: [E-devel] E SVN: andreas trunk/e/src/modules/conf_screensaver

2009-04-14 Thread Andreas Volz
Am Fri, 10 Apr 2009 22:54:45 -0300 schrieb Gustavo Sverzut Barbieri: > On Fri, Apr 10, 2009 at 7:34 PM, Enlightenment SVN > wrote: > > >  - now used Eina_List for storage (I hope I used it correct...) > > > +   Eina_List *l = NULL; > > +   Evas_Object *o = NULL; > > + > > +   // delete the list

Re: [E-devel] E SVN: andreas trunk/e/src/modules/conf_screensaver

2009-04-10 Thread Gustavo Sverzut Barbieri
On Fri, Apr 10, 2009 at 7:34 PM, Enlightenment SVN wrote: >  - now used Eina_List for storage (I hope I used it correct...) > +   Eina_List *l = NULL; > +   Evas_Object *o = NULL; > + > +   // delete the list > +   for (l = xscreensaver_list; l; l = eina_list_next(l)) > +   { > +      xscreensav