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.

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

Reply via email to