[mythtv] [PATCH] Autoexpire blocked by missing file.

Chris Pinkham cpinkham at bc2va.org
Sun Mar 13 21:28:44 UTC 2005


> Here is my updated patch.  In addition to the changes we discussed
> earlier I created a new function, AutoExpire::SelectFile, which
> contains the bulk of my changes.

Thanks.  This is committed with a minor mod.  The reason that I had
the expireList sorted in reverse order originally was because the code
used to delete as many recordings as necessary in one trip through the main
loop.  Then when a couple people had things go wrong on their system, the
code was changed to only delete one recording per loop, so if it needs to
delete 2 recordings, it takes 2 loops (with the sleep in the middle).
Since we now only delete one recording per loop and refill the expireList
every time through the loop, I modified the code to put the expireList in
order to we delete off the front.  This makes the logic in the code easier
since we don't need to test for i == expireList.begin() and handle that
situation and it also means we can exit SelectFile right away when we find
an item we can expire.  Also changed a couple other things to do with the
expireList ordering.  I figured it would make the code cleaner to keep
the list in expire order instead of reverse order.

-- 

Chris



More information about the mythtv-dev mailing list