2015-01-22 11:13 GMT+01:00 Daniel Zaoui <daniel.za...@samsung.com>:

> Yo,
>
> On 01/22/15 11:58, Davide Andreoli wrote:
> > 2015-01-21 23:11 GMT+01:00 Cedric BAIL <cedric.b...@free.fr>:
> >
> >> On Wed, Jan 21, 2015 at 10:02 PM, Davide Andreoli
> >> <d...@gurumeditation.it> wrote:
> >>> and new api's in feature freeze time?
> >> This one is not an new API. The error message, we were seeing, was due
> >> to the fact that elm_image only partially implemented the redirection
> >> of Edje_Object leading to unimplemented call that resulted in previous
> >> issue. This solve that. The implementation is pretty straight forward
> >> and seems ok to me.
> >>
> > Ok, thanks for the explanation and sorry for bother then.
> >
> > All this Eo stuff is still not really clear to me... doesn't the added
> > 4 "implements" result in eo to autogenerate 4 new legacy functions?
> > elm_image_size_min_get(...)
> > elm_image_size_min_set(...)
> > etc..
>
> If the functions are declared inside elm_image.eo, legacy will be
> generated. If elm_image implement some interface and its functions, no
> legacy functions will be added. There is no reason for that addition
> cause you already have the legacy API of the interface itself, e.g
> edje_object_size_min_get. I hope I was enough clear :-)
>

Yep, thanks, clear now :)


>
> >
> >
> >
> >
> >> To be precise, when you swallow an object that provide an Edje_Object
> >> interface we expect those 4 functions to be there and implemented. If
> >> it is not, swallowing a pseudo edje object will more or less fail :-)
> >>
> >>> 2015-01-21 21:17 GMT+01:00 Mike Blumenkrantz <zm...@osg.samsung.com>:
> >>>
> >>>> discomfitor pushed a commit to branch master.
> >>>>
> >>>>
> >>>>
> >>
> http://git.enlightenment.org/core/elementary.git/commit/?id=c04d793dec1063a6dae1c3f403c56d8df1607303
> >>>> commit c04d793dec1063a6dae1c3f403c56d8df1607303
> >>>> Author: Mike Blumenkrantz <zm...@osg.samsung.com>
> >>>> Date:   Wed Jan 21 15:15:43 2015 -0500
> >>>>
> >>>>     image now implements edje object size functions
> >>>>
> >>>>           Edje.Object.size_min.get;
> >>>>           Edje.Object.size_max.get;
> >>>>           Edje.Object.size_min_calc;
> >>>>           Edje.Object.calc_force;
> >>>>
> >>>>     fixes all image-in-edje sizing issues/errors
> >>>> ---
> >>>>  src/lib/elm_image.c  | 34 ++++++++++++++++++++++++++++++++++
> >>>>  src/lib/elm_image.eo |  4 ++++
> >>>>  2 files changed, 38 insertions(+)
> >>>>
> >>>> diff --git a/src/lib/elm_image.c b/src/lib/elm_image.c
> >>>> index 8e12a39..4ca45fa 100644
> >>>> --- a/src/lib/elm_image.c
> >>>> +++ b/src/lib/elm_image.c
> >>>> @@ -841,6 +841,40 @@ _elm_image_edje_object_signal_emit(Eo *obj
> >>>> EINA_UNUSED, Elm_Image_Data *sd, cons
> >>>>       edje_object_signal_emit(sd->img, emission, source);
> >>>>  }
> >>>>
> >>>> +EOLIAN static void
> >>>> +_elm_image_edje_object_size_min_get(Eo *obj EINA_UNUSED,
> Elm_Image_Data
> >>>> *sd, int *w, int *h)
> >>>> +{
> >>>> +   if (sd->edje)
> >>>> +     edje_object_size_min_get(sd->img, w, h);
> >>>> +   else
> >>>> +     evas_object_size_hint_min_get(sd->img, w, h);
> >>>> +}
> >>>> +
> >>>> +EOLIAN static void
> >>>> +_elm_image_edje_object_size_max_get(Eo *obj EINA_UNUSED,
> Elm_Image_Data
> >>>> *sd, int *w, int *h)
> >>>> +{
> >>>> +   if (sd->edje)
> >>>> +     edje_object_size_max_get(sd->img, w, h);
> >>>> +   else
> >>>> +     evas_object_size_hint_max_get(sd->img, w, h);
> >>>> +}
> >>>> +
> >>>> +EOLIAN static void
> >>>> +_elm_image_edje_object_calc_force(Eo *obj EINA_UNUSED, Elm_Image_Data
> >> *sd)
> >>>> +{
> >>>> +   if (sd->edje)
> >>>> +     edje_object_calc_force(sd->img);
> >>>> +}
> >>>> +
> >>>> +EOLIAN static void
> >>>> +_elm_image_edje_object_size_min_calc(Eo *obj EINA_UNUSED,
> >> Elm_Image_Data
> >>>> *sd, int *w, int *h)
> >>>> +{
> >>>> +   if (sd->edje)
> >>>> +     edje_object_size_min_calc(sd->img, w, h);
> >>>> +   else
> >>>> +     evas_object_size_hint_min_get(sd->img, w, h);
> >>>> +}
> >>>> +
> >>>>  EOLIAN static Eina_Bool
> >>>>  _elm_image_mmap_set(Eo *obj, Elm_Image_Data *sd, const Eina_File *f,
> >>>> const char *key)
> >>>>  {
> >>>> diff --git a/src/lib/elm_image.eo b/src/lib/elm_image.eo
> >>>> index ef934d4..d5592cb 100644
> >>>> --- a/src/lib/elm_image.eo
> >>>> +++ b/src/lib/elm_image.eo
> >>>> @@ -543,6 +543,10 @@ class Elm_Image (Elm_Widget, Efl.File, Efl.Image,
> >>>> Evas.Clickable_Interface,
> >>>>        Efl.Image.smooth_scale.set;
> >>>>        Efl.Image.smooth_scale.get;
> >>>>        Edje.Object.signal_emit;
> >>>> +      Edje.Object.size_min.get;
> >>>> +      Edje.Object.size_max.get;
> >>>> +      Edje.Object.size_min_calc;
> >>>> +      Edje.Object.calc_force;
> >>>>        Evas.Object_Smart.hide;
> >>>>        Evas.Object_Smart.clip.set;
> >>>>        Evas.Object_Smart.clip_unset;
> >>>>
> >>>> --
> >>>>
> >>>>
> >>>>
> >>
> ------------------------------------------------------------------------------
> >>> New Year. New Location. New Benefits. New Data Center in Ashburn, VA.
> >>> GigeNET is offering a free month of service with a new server in
> Ashburn.
> >>> Choose from 2 high performing configs, both with 100TB of bandwidth.
> >>> Higher redundancy.Lower latency.Increased capacity.Completely
> compliant.
> >>> http://p.sf.net/sfu/gigenet
> >>> _______________________________________________
> >>> enlightenment-devel mailing list
> >>> enlightenment-devel@lists.sourceforge.net
> >>> https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
> >>>
> >>
> >>
> >> --
> >> Cedric BAIL
> >>
> >>
> >>
> ------------------------------------------------------------------------------
> >> New Year. New Location. New Benefits. New Data Center in Ashburn, VA.
> >> GigeNET is offering a free month of service with a new server in
> Ashburn.
> >> Choose from 2 high performing configs, both with 100TB of bandwidth.
> >> Higher redundancy.Lower latency.Increased capacity.Completely compliant.
> >> http://p.sf.net/sfu/gigenet
> >> _______________________________________________
> >> enlightenment-devel mailing list
> >> enlightenment-devel@lists.sourceforge.net
> >> https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
> >>
> >
> ------------------------------------------------------------------------------
> > New Year. New Location. New Benefits. New Data Center in Ashburn, VA.
> > GigeNET is offering a free month of service with a new server in Ashburn.
> > Choose from 2 high performing configs, both with 100TB of bandwidth.
> > Higher redundancy.Lower latency.Increased capacity.Completely compliant.
> > http://p.sf.net/sfu/gigenet
> > _______________________________________________
> > enlightenment-devel mailing list
> > enlightenment-devel@lists.sourceforge.net
> > https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
> >
>
>
------------------------------------------------------------------------------
New Year. New Location. New Benefits. New Data Center in Ashburn, VA.
GigeNET is offering a free month of service with a new server in Ashburn.
Choose from 2 high performing configs, both with 100TB of bandwidth.
Higher redundancy.Lower latency.Increased capacity.Completely compliant.
http://p.sf.net/sfu/gigenet
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to