On Wed, 23 Dec 2009, Jose Gonzalez wrote:
> Vincent Torri wrote: > >> >> On Tue, 22 Dec 2009, Jose Gonzalez wrote: >> >>> Vincent Torri wrote: >>> >>>> >>>> the correct solution is to fix the incredible mess of the ecore headers. >>>> >>> >>> Any concrete suggestions on exactly how to "fix" that? >> >> the first thing to do is to replace in Ecore.h all the >> >> typedef void Ecore_****; >> >> by the correct: >> >> typedef struct Ecore_*** Ecore_***; >> >> Then, remove in Ecore.h the >> >> #ifndef _ECORE_PRIVATE_H >> >> Then, remove in ecore_private.h all the stuff that is also in Ecore.h (now, >> you can remark that, for example, the definition of enum _Ecore_Exe_Flags >> is in both files). >> >> To be sure that everything is fine, reverse the order of inclusion of >> Ecore.h and ecore_private.h (include first ecore_private.h, then Ecore.h). >> If there are other things to fix, there will be an error with that order. >> >> Those are the first steps. Because these changes implies more things to do >> in other ecore files. >> >> What i would like is a private header file for each source file instead of >> a big ecore_private.h. It's a bit more work in the ecore subsystem, but it >> should save work in other ecore_* subsystems, I think. What are mainly used >> in the other subsystems are events, timers, animators, handlers, that is, >> what must be managed in the main loop. >> > > > Well, if you feel you can make this work fine and no one raises any > objections or problems, then maybe you should give it a shot.. at least your > 'first steps' part doesn't seem like much work. Anything else? It seems that Seb already did the job. The other remaining stuff ? The other private header files are also a mess... It's not hard. It's just boring Vincent ------------------------------------------------------------------------------ This SF.Net email is sponsored by the Verizon Developer Community Take advantage of Verizon's best-in-class app development support A streamlined, 14 day to market process makes app distribution fast and easy Join now and get one step closer to millions of Verizon customers http://p.sf.net/sfu/verizon-dev2dev _______________________________________________ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
