[mythtv] frequent PlayBoxTransparency setting DB lookup in mythvideo

Donavan_Stanley at ham.honda.com Donavan_Stanley at ham.honda.com
Tue Aug 17 10:34:49 EDT 2004


Again:  Replying from work with a crappy client.  I apologize.


"I dunno if this will be just in MythVideo or if it'll come out to
libmyth too, but if you're commiting to MythVideo, do you fancy taking a
quick look at the patch in my email titled "[mythtv] [Partial PATCH+BUG
Report] MythVideo high CPU usage on Browser & freeze on Gallery" sent to
the dev list yesterday (16th) at 22:54 BST (GMT +0100).

There isn't much in it, but there is a couple of bugs addressed. Feel
free to be selective!"





Pretty much everything I'm doing is in MythVideo, except for a minor change
to the UIImageType so that I can get the filename of the image it currently
has loaded.  I'm changing VideoDialog to be MythDialog derrived instead of
MythThemeDialog.  With that in and the changes I made to how the cover art
gets handled in places things should be much snappier when drawing.


I did look at your patch and had planned on applying at least part of it.
There was one line that seemed to be wrong, there was a check for the
failure of a query that used "rowsaffected == 1" when it seemed like it
should have used "rowsaffected < 1".


I also wondered why you tightened the checks for the queries from ">0" to
"==1".   It shouldn't really impact things one way or another, I just
didn't understand the reasoning behind it.


 Oh and, there's no need to use QString.sprintf when doing a simple
assignment.  There's no sense in parsing the string for replacement chars
if we know there aren't any.  Actually is there a real need to create a
QString out of that literal string at all?




More information about the mythtv-dev mailing list