Ping? Anyone?

On Fri, Nov 18, 2016 at 04:18:00PM +0100, marcel-hollerb...@t-online.de wrote:
> Hello,
> 
> i would like to get back to this.
> 
> On Tue, Nov 01, 2016 at 11:48:50AM -0700, Jee-Yong Um wrote:
> > cedric pushed a commit to branch master.
> > 
> > http://git.enlightenment.org/core/efl.git/commit/?id=0fd144550877ff0a6f413c025685b7e7d63e6535
> > 
> > commit 0fd144550877ff0a6f413c025685b7e7d63e6535
> > Author: Jee-Yong Um <con...@gmail.com>
> > Date:   Tue Nov 1 10:59:09 2016 -0700
> > 
> >     edje.object: implement Efl.Observer interface
> >     
> >     Summary:
> >     To remove duplicated lines to handle edje class (color, text, size),
> >     observer interface is implemented to Edje.Object.
> >     
> >     Reviewers: jpeg, cedric
> >     
> >     Reviewed By: cedric
> >     
> >     Subscribers: bu5hm4n, cedric
> >     
> >     Differential Revision: https://phab.enlightenment.org/D4359
> >     
> >     Signed-off-by: Cedric BAIL <ced...@osg.samsung.com>
> > ---
> >  src/Makefile_Efl.am                      |   3 +
> >  src/lib/edje/edje_load.c                 |   8 +-
> >  src/lib/edje/edje_main.c                 |  45 +++-
> >  src/lib/edje/edje_object.eo              |   4 +-
> >  src/lib/edje/edje_private.h              |  19 +-
> >  src/lib/edje/edje_smart.c                |  50 ++++
> >  src/lib/edje/edje_text.c                 |   8 +-
> >  src/lib/edje/edje_textblock_styles.c     |   6 +-
> >  src/lib/edje/edje_util.c                 | 441 
> > ++++---------------------------
> >  src/lib/efl/Efl.h                        |   3 +
> >  src/lib/efl/interfaces/efl_observable.eo |  63 +++++
> >  src/lib/efl/interfaces/efl_observer.c    | 259 ++++++++++++++++++
> >  src/lib/efl/interfaces/efl_observer.eo   |  15 ++
> >  src/lib/efl/interfaces/efl_types.eot     |   6 +
> >  14 files changed, 514 insertions(+), 416 deletions(-)
> >
>  [snip]
> > +#include "efl_observable.eo.c"
> > +#include "efl_observer.eo.c"
> > diff --git a/src/lib/efl/interfaces/efl_observer.eo 
> > b/src/lib/efl/interfaces/efl_observer.eo
> > new file mode 100644
> > index 0000000..a0d4ffe
> > --- /dev/null
> > +++ b/src/lib/efl/interfaces/efl_observer.eo
> > @@ -0,0 +1,15 @@
> > +interface Efl.Observer {
> > +   methods {
> > +      update @virtual_pure {
> > +         [[Update observer according to the changes of observable object.
> > +
> > +           @since 1.19]]
> > +         params {
> > +            /* FIXME: obs should be Efl.Observable, but cyclic dependency 
> > error occurs. */
> > +            @in obs: Efl.Object; [[An observable object]]
> 
> It looks like this is never going to change. So i guess its quite
> dangerous to leave this interface like this, since its not going to work
> like this. And the interface itself probebly needs to be defined in a
> way that it works with the way eolian handles cyclic dependencies
> 
> Greetings
>    Marcel Hollerbach
>  
> 
> ------------------------------------------------------------------------------
> _______________________________________________
> enlightenment-devel mailing list
> enlightenment-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most 
engaging tech sites, SlashDot.org! http://sdm.link/slashdot
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to