On Mon, Jan 7, 2013 at 9:48 PM, Gustavo Lima Chaves <gl...@profusion.mobi>wrote:

> * Enlightenment SVN <no-re...@enlightenment.org> [2013-01-04 21:59:18
> -0800]:
>
> > Log:
> > elementary/label - removed unnecessary
> >
> >
> >
> > Author:       hermet
> > Date:         2013-01-04 21:59:18 -0800 (Fri, 04 Jan 2013)
> > New Revision: 82266
> > Trac:         http://trac.enlightenment.org/e/changeset/82266
> >
> > Modified:
> >   trunk/elementary/src/lib/elm_label.c
> >
> > Modified: trunk/elementary/src/lib/elm_label.c
> > ===================================================================
> > --- trunk/elementary/src/lib/elm_label.c      2013-01-05 05:50:13 UTC
> (rev 82265)
> > +++ trunk/elementary/src/lib/elm_label.c      2013-01-05 05:59:18 UTC
> (rev 82266)
> > @@ -643,7 +643,6 @@
> >  EINA_DEPRECATED EAPI Eina_Bool
> >  elm_label_slide_get(const Evas_Object *obj)
> >  {
> > -   ELM_LABEL_CHECK(obj) EINA_FALSE;
>
> Hi, Hermet.
> I don't get why the magic check is unnecessary.
>

I think he removed it because magic is checked in
elm_label_slide_mode_get().
But explicitly checking magic in EAPI looks better.

Daniel Juyung Seo (SeoZ)


> >     Eina_Bool ret = EINA_FALSE;
> >     if (elm_label_slide_mode_get(obj) == ELM_LABEL_SLIDE_MODE_ALWAYS)
> >       ret = EINA_TRUE;
> >
> >
> >
> ------------------------------------------------------------------------------
> > Master Visual Studio, SharePoint, SQL, ASP.NET, C# 2012, HTML5, CSS,
> > MVC, Windows 8 Apps, JavaScript and much more. Keep your skills current
> > with LearnDevNow - 3,200 step-by-step video tutorials by Microsoft
> > MVPs and experts. SALE $99.99 this month only -- learn more at:
> > http://p.sf.net/sfu/learnmore_122912
> > _______________________________________________
> > enlightenment-svn mailing list
> > enlightenment-...@lists.sourceforge.net
> > https://lists.sourceforge.net/lists/listinfo/enlightenment-svn
>
> --
> Gustavo Lima Chaves
> Senior Developer @ ProFUSION Embedded Systems
>
>
> ------------------------------------------------------------------------------
> Master Visual Studio, SharePoint, SQL, ASP.NET, C# 2012, HTML5, CSS,
> MVC, Windows 8 Apps, JavaScript and much more. Keep your skills current
> with LearnDevNow - 3,200 step-by-step video tutorials by Microsoft
> MVPs and experts. SALE $99.99 this month only -- learn more at:
> http://p.sf.net/sfu/learnmore_122412
> _______________________________________________
> enlightenment-devel mailing list
> enlightenment-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
>
------------------------------------------------------------------------------
Master Visual Studio, SharePoint, SQL, ASP.NET, C# 2012, HTML5, CSS,
MVC, Windows 8 Apps, JavaScript and much more. Keep your skills current
with LearnDevNow - 3,200 step-by-step video tutorials by Microsoft
MVPs and experts. SALE $99.99 this month only -- learn more at:
http://p.sf.net/sfu/learnmore_122412
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to