[mythtv] Patch: Small cleanup in mythfilldatabase

Ed Wildgoose lists at wildgooses.com
Thu Apr 8 18:07:01 EDT 2004


This is a small fix that I thought I had submitted some time back to 
finish the integration for the tv_grab_uk_rt grabber.  It's needed for 
the --quiet flag to work properly - it wasnt added in the original patch 
because "--output" was not an option for this grabber.

There is also an easy tweak to catch the case that there can be no real 
data for a day, but there can be the odd entry filled in here and there 
(perhaps due to minor overrun from the day before).  I think there 
should be no sideeffects here

Ed W
-------------- next part --------------
Index: programs/mythfilldatabase/filldata.cpp
===================================================================
RCS file: /var/lib/mythcvs/mythtv/programs/mythfilldatabase/filldata.cpp,v
retrieving revision 1.96
diff -u -r1.96 filldata.cpp
--- programs/mythfilldatabase/filldata.cpp	14 Mar 2004 07:35:32 -0000	1.96
+++ programs/mythfilldatabase/filldata.cpp	8 Apr 2004 21:49:58 -0000
@@ -1438,7 +1438,7 @@
                         xmltv_grabber.ascii(), configfile.ascii(), 
                         filename.ascii());
     else if (xmltv_grabber == "tv_grab_uk_rt")
-        command.sprintf("nice %s --days 1 --offset %d --config-file '%s' > %s",
+        command.sprintf("nice %s --days 1 --offset %d --config-file '%s' --output %s",
 	                 xmltv_grabber.ascii(), offset, 
 			 configfile.ascii(), filename.ascii());
     else if (xmltv_grabber == "tv_grab_au")
@@ -1749,8 +1749,10 @@
                
                 if (query.isActive()) 
                 {
-                    if (!query.numRowsAffected() ||
-                        (query.next() && query.value(0).toInt() <= 1)) 
+		    // We also need to get this days data if there is 
+		    // only a suspiciously small amount in the DB
+                    if (!query.numRowsAffected() || 
+                        (query.next() && query.value(0).toInt() <= 20)) 
                     {
                         download_needed = true;
                     }


More information about the mythtv-dev mailing list