[mythtv-commits] mythtv commit: r6989 by danielk

mythtv at cvs.mythtv.org mythtv at cvs.mythtv.org
Thu Aug 4 20:14:41 UTC 2005


      Author: danielk
        Date: 2005-08-04 20:14:41 +0000 (Thu, 04 Aug 2005)
New Revision: 6989
   Changeset: http://cvs.mythtv.org/trac/changeset/6989

Modified:

   trunk/mythtv/libs/libmythtv/tv_play.cpp

Log:

Another small memory problem, this one has to do with
QDateTime::secsTo() using a reference and never assigning
the QDateTime::currentTime() to a variable.

Whether this should work according the C++ standard,
I'm don't know. I think the compiler implementers
should keep the currentTime() return value around while
it is being referenced and should stick it in a temporary
memory location when pass-by-reference was used....

But I do know this does cause bad code to be generated
with gcc 3.3.5 at "-O0"...






More information about the mythtv-commits mailing list