2015-04-15 10:06 GMT+02:00 Yakov Goldberg <yako...@samsung.com>:

> On 04/15/2015 09:32 AM, Stefan Schmidt wrote:
> > Hello.
> >
> > On 14/04/15 22:57, Dave Andreoli wrote:
> >> davemds pushed a commit to branch master.
> >>
> >>
> http://git.enlightenment.org/core/elementary.git/commit/?id=d021b7e36d001ecdeb3767bb6a67cc361d0f7ee9
> >>
> >> commit d021b7e36d001ecdeb3767bb6a67cc361d0f7ee9
> >> Author: Dave Andreoli <d...@gurumeditation.it>
> >> Date:   Tue Apr 14 22:19:18 2015 +0200
> >>
> >>      Add missing file_get legacy definition for Video and Layout
> >>
> >>      was missed from commit:
> >>      e5cda41c889f98b6f3f5f63b909f595a7a656cc0
> > If these legacy defines have only been forgotten it is totally fine to
> > add them now. I just want to hear from Yakov if he really forgot or if
> > he had a reason not adding them but forgot to say so in the commit
> > message. Yakov?
> I didn't add these legace APIs because they have never existed.
> I just added FILE interface implementations for Layout and Video to
> avoid error message.
> So do we need them now?
>

We have the setter definition in the legacy api, why not the getter?
Is there a reason?


>
> >
> > regards
> > Stefan Schmidt
> >
> >
> >> ---
> >>   src/lib/elm_layout.c        |  6 ++++++
> >>   src/lib/elm_layout_legacy.h | 13 +++++++++++++
> >>   src/lib/elm_video.c         |  6 ++++++
> >>   src/lib/elm_video_legacy.h  | 12 ++++++++++++
> >>   4 files changed, 37 insertions(+)
> >>
> >> diff --git a/src/lib/elm_layout.c b/src/lib/elm_layout.c
> >> index 19f6b47..f04b80f 100644
> >> --- a/src/lib/elm_layout.c
> >> +++ b/src/lib/elm_layout.c
> >> @@ -1725,4 +1725,10 @@ elm_layout_file_set(Eo *obj, const char *file,
> const char *group)
> >>      return eo_do_ret((Eo *) obj, ret, efl_file_set(file, group));
> >>   }
> >>
> >> +EAPI void
> >> +elm_layout_file_get(Eo *obj, const char **file, const char **group)
> >> +{
> >> +   eo_do((Eo *) obj, efl_file_get(file, group));
> >> +}
> >> +
> >>   #include "elm_layout.eo.c"
> >> diff --git a/src/lib/elm_layout_legacy.h b/src/lib/elm_layout_legacy.h
> >> index 1df030d..e5bd857 100644
> >> --- a/src/lib/elm_layout_legacy.h
> >> +++ b/src/lib/elm_layout_legacy.h
> >> @@ -88,4 +88,17 @@ EAPI Evas_Object
>  *elm_layout_content_unset(Evas_Object *obj, con
> >>    */
> >>   EAPI Eina_Bool elm_layout_file_set(Eo *obj, const char *file, const
> char *group);
> >>
> >> +/**
> >> + * Get the loaded file
> >> + *
> >> + *
> >> + * @ingroup Layout
> >> + *
> >> + * @param file The path to file (edj) used as layout
> >> + * @param group The group that the layout belongs in edje file
> >> + *
> >> + * @since 1.14
> >> + */
> >> +EAPI void elm_layout_file_get(Eo *obj, const char **file, const char
> **group);
> >> +
> >>   #include "elm_layout.eo.legacy.h"
> >> diff --git a/src/lib/elm_video.c b/src/lib/elm_video.c
> >> index b90e3f9..56f8ad0 100644
> >> --- a/src/lib/elm_video.c
> >> +++ b/src/lib/elm_video.c
> >> @@ -460,5 +460,11 @@ elm_video_file_set(Eo *obj, const char *filename)
> >>      return eo_do_ret((Eo *) obj, ret, efl_file_set(filename, NULL));
> >>   }
> >>
> >> +EAPI void
> >> +elm_video_file_get(Eo *obj, const char **filename)
> >> +{
> >> +   eo_do((Eo *) obj, efl_file_get(filename, NULL));
> >> +}
> >> +
> >>
> >>   #include "elm_video.eo.c"
> >> diff --git a/src/lib/elm_video_legacy.h b/src/lib/elm_video_legacy.h
> >> index 82e4191..184d31a 100644
> >> --- a/src/lib/elm_video_legacy.h
> >> +++ b/src/lib/elm_video_legacy.h
> >> @@ -48,4 +48,16 @@ the available Webcam on your system).
> >>    */
> >>   EAPI Eina_Bool elm_video_file_set(Eo *obj, const char *filename);
> >>
> >> +/**
> >> + *
> >> + * @brief Get the file or URI that is used as the video source.
> >> + *
> >> + * @ingroup Video
> >> + *
> >> + * @param filename The file or URI.
> >> + *
> >> + * @since 1.14
> >> + */
> >> +EAPI void elm_video_file_get(Eo *obj, const char **filename);
> >> +
> >>   #include "elm_video.eo.legacy.h"
> >>
> >
>
>
>
> ------------------------------------------------------------------------------
> BPM Camp - Free Virtual Workshop May 6th at 10am PDT/1PM EDT
> Develop your own process in accordance with the BPMN 2 standard
> Learn Process modeling best practices with Bonita BPM through live
> exercises
> http://www.bonitasoft.com/be-part-of-it/events/bpm-camp-virtual-
> event?utm_
> source=Sourceforge_BPM_Camp_5_6_15&utm_medium=email&utm_campaign=VA_SF
> _______________________________________________
> enlightenment-devel mailing list
> enlightenment-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
>
------------------------------------------------------------------------------
BPM Camp - Free Virtual Workshop May 6th at 10am PDT/1PM EDT
Develop your own process in accordance with the BPMN 2 standard
Learn Process modeling best practices with Bonita BPM through live exercises
http://www.bonitasoft.com/be-part-of-it/events/bpm-camp-virtual- event?utm_
source=Sourceforge_BPM_Camp_5_6_15&utm_medium=email&utm_campaign=VA_SF
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to