[Gnash-dev] Re: [Gnash-commit] [SCM] Gnash branch, master, updated. 6895d3b0542e1ac28a3e37dd150187dacb66195d

[John Gilmore]

> --- a/libmedia/MediaParser.cpp
> +++ b/libmedia/MediaParser.cpp
> @@ -411,6 +411,12 @@ MediaParser::parserLoop()
>       while (!parserThreadKillRequested())
>       {
>               parseNextChunk();
> +             gnashSleep(100); // thread switch 
> +
> +             // check for parsing complete
> +             // TODO: have a setParsingComplete() function
> +             //       exposed in base class for taking care
> +             //       of this on appropriate time.
>               boost::mutex::scoped_lock lock(_qMutex);
>               waitIfNeeded(lock);
>       }

If you wanted to revert the change, this won't do it, because we are
still doing waitIfNeeded both in the main loop, and in each of the
audio and video parseNextChunk's.

There should be no need for this gnashSleep.  If we have our locks
and semaphores set correctly, the system scheduler should do the job
without being manually forced to switch tasks.

        John