[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