> 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
_______________________________________________ mythtv-dev mailing list [email protected] http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-dev
