[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Gnash-commit] gnash ChangeLog server/movie_root.cpp testsuite...
From: |
zou lunkai |
Subject: |
Re: [Gnash-commit] gnash ChangeLog server/movie_root.cpp testsuite... |
Date: |
Tue, 25 Dec 2007 16:59:28 +0800 |
Ah, you might be confused by the cvs log:) Don't read that, check
the source directly. Those code are not part of
movie_root::flushHigherPriorityActionQueues().
--zou
On Dec 25, 2007 4:55 PM, zou lunkai <address@hidden> wrote:
> On Dec 25, 2007 4:48 PM, strk <address@hidden> wrote:
> > On Tue, Dec 25, 2007 at 07:47:10AM +0000, Zou Lunkai wrote:
> >
> > > + if( ! _intervalTimers.empty() )
> > > + {
> > > + // process actions queued when executing interval callbacks
> > > + processActionQueue();
> > > + }
> > > +
> > > }
> >
> > The above doesn't seem correct.
> > The fact that _intervalTimers is not empty doesn't mean any was
> > actually expired...
> > Better have executeTimers return a value to notify whether or not
> > any was expired.
> >
>
> Logicallly, this check is not even needed. It is just for optimizing.
> ie. if there is no interval callbacks, no actions would be queued,
> thus we don't need to process any actions within
> 'movie_root::executeTimers()' function body.
>
> I am not sure what you mean.
>
> --zou
>