On Tue, 18 Sep 2012 13:40:01 +0200 Vincent Torri <vincent.to...@gmail.com> said:

> On Tue, Sep 18, 2012 at 1:38 PM, Enlightenment SVN
> <no-re...@enlightenment.org> wrote:
> > Modified: trunk/ecore/src/lib/ecore/ecore_timer.c
> > ===================================================================
> > --- trunk/ecore/src/lib/ecore/ecore_timer.c     2012-09-18 11:34:45 UTC
> > (rev 76804) +++ trunk/ecore/src/lib/ecore/ecore_timer.c     2012-09-18
> > 11:38:08 UTC (rev 76805) @@ -233,6 +233,12 @@
> >     EINA_MAIN_LOOP_CHECK_RETURN;
> >     _ecore_lock();
> 
> why after _ecore_lock() ?
> 
> Vincent

doesn't really matter.. i guess it can be less code if before.

> > +   if (in < 0.0)
> > +     {
> > +        ERR("Interval %f less than zero, ignored", in);
> > +        goto unlock;
> > +     }
> > +
> >     if (!ECORE_MAGIC_CHECK(timer, ECORE_MAGIC_TIMER))
> >       {
> >          ECORE_MAGIC_FAIL(timer, ECORE_MAGIC_TIMER,
> >
> >
> > ------------------------------------------------------------------------------
> > Live Security Virtual Conference
> > Exclusive live event will cover all the ways today's security and
> > threat landscape has changed and how IT managers can respond. Discussions
> > will include endpoint security, mobile security and the latest in malware
> > threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
> > _______________________________________________
> > enlightenment-svn mailing list
> > enlightenment-...@lists.sourceforge.net
> > https://lists.sourceforge.net/lists/listinfo/enlightenment-svn
> 
> ------------------------------------------------------------------------------
> Live Security Virtual Conference
> Exclusive live event will cover all the ways today's security and 
> threat landscape has changed and how IT managers can respond. Discussions 
> will include endpoint security, mobile security and the latest in malware 
> threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
> _______________________________________________
> enlightenment-devel mailing list
> enlightenment-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
> 


-- 
------------- Codito, ergo sum - "I code, therefore I am" --------------
The Rasterman (Carsten Haitzler)    ras...@rasterman.com


------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
will include endpoint security, mobile security and the latest in malware 
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to