[mythtv] Re: [mythtv-commits] mythtv commits

Isaac Richards ijr at case.edu
Sat Apr 9 19:40:39 UTC 2005


On Saturday 09 April 2005 03:30 pm, Daniel Kristjansson wrote:
> There are three solutions as I see it:
> 1/ Construct the verbose string completely outside of the verbose lock.
>    The problem with this is it makes the verbose macro take longer to
>    execute.

Right, would:

ostringstream blah;
blah << dtime << " " << args;
mutex.lock();
cout << blah << endl;
mutex.unlock();

Really take that much extra time?

> 2/ Examine every VERBOSE macro with redirects in it and either check if
>    it is calling a blocking function, or just transform it to use
>    QString().arg().arg().. Problem: lots of developer time..

Lots and lots..  So, not going to happen. =)

> 3/ Disable verbose locking in release builds.

This would work, too.

Isaac


More information about the mythtv-dev mailing list