[mythtv] [patch] problem with mythfilldatabase

Kenneth Aafløy ke-aa at frisurf.no
Sat Jan 24 18:32:48 EST 2004


Hi,

My own ego wrote:
> Dave Alden wrote:
> > I noticed my last couple of runs of mythfilldatabase have 
> > been erroring out.  I tracked it down to the patch for
> > filldata.cpp (version 1.82) that added the following lines:
> > 
> >      from.setDate(*qCurrentDate);
> >      from.addDays(offset);
> >      from.addSecs(listing_wrap_offset);
> >      to = from;
> >      to.addDays(nextoffset);
> > 
> > The 2 lines that call addDays should have reassigned the 
> > value to the variable.  I've attached a patch.  :-)
> 
> Thanks for clearing up my mess, better reassign the 
> from.addSecs(x) too then =)

Please add this patch also.

Should the date be reassigned as is done in the for loop?

Kenneth

Index: filldata.cpp
===================================================================
RCS file:
/var/lib/mythcvs/mythtv/programs/mythfilldatabase/filldata.cpp,v
retrieving revision 1.82
diff -u -r1.82 filldata.cpp
--- filldata.cpp        22 Jan 2004 03:56:14 -0000      1.82
+++ filldata.cpp        24 Jan 2004 23:29:49 -0000
@@ -1568,18 +1568,24 @@
 
             if (refresh_today)
             {
+                if (!quiet)
+                    cout << "Refreshing Todays data" << endl;
                 if (!grabData(*it, 0, &qCurrentDate))
                     ++failures;
             }
 
             if (refresh_tomorrow)
             {
+                if (!quiet)
+                    cout << "Refreshing Tomorrows data" << endl;
                 if (!grabData(*it, 1, &qCurrentDate))
                     ++failures;
             }
 
             if (refresh_second)
             {
+                if (!quiet)
+                    cout << "Refreshing data for 2 days from today" <<
endl;
                 if (!grabData(*it, 2, &qCurrentDate))
                     ++failures;
             }



More information about the mythtv-dev mailing list