From: Dave A. <da...@gu...> - 2009-09-05 00:34:27
|
2009/9/4 Enlightenment SVN <no-...@en...> > Log: > e17:shutdown all modules > > Author: jeffdameth > Date: 2009-09-04 12:43:10 -0700 (Fri, 04 Sep 2009) > New Revision: 42262 > > Modified: > trunk/e/src/bin/e_module.c > > Modified: trunk/e/src/bin/e_module.c > =================================================================== > --- trunk/e/src/bin/e_module.c 2009-09-04 19:27:12 UTC (rev 42261) > +++ trunk/e/src/bin/e_module.c 2009-09-04 19:43:10 UTC (rev 42262) > @@ -37,7 +37,8 @@ > e_module_shutdown(void) > { > E_Module *m; > - > + Eina_List *l; > + > #ifdef HAVE_VALGRIND > /* do a leak check now before we dlclose() all those plugins, cause > * that means we won't get a decent backtrace to leaks in there > @@ -45,10 +46,15 @@ > VALGRIND_DO_LEAK_CHECK > #endif > > - _e_modules = eina_list_reverse(_e_modules); > - > - EINA_LIST_FREE(_e_modules, m) > + // whats was this for? modules should not depend on each others load > order > + // _e_modules = eina_list_reverse(_e_modules); > Loading (and unloading) order is important!! I have done this for the gadman module, than need to be loaded after all the other modules. The order is controlled by e_module_priority_set(). DaveMDS > + > + /* FIXME there seems to be a module that modifies _e_modules here > + * dont use EINA_LIST_FREE until solved! */ > + EINA_LIST_FOREACH(_e_modules, l, m) > { > + printf("shutdown module: %s\n", m->name); > + > if (m && m->enabled && !m->error) > { > m->func.save(m); > @@ -58,6 +64,10 @@ > e_object_del(E_OBJECT(m)); > } > > + if (_e_modules) > + eina_list_free(_e_modules); > + _e_modules = NULL; > + > return 1; > } > > > > > ------------------------------------------------------------------------------ > 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-svn mailing list > enl...@li... > https://lists.sourceforge.net/lists/listinfo/enlightenment-svn > |