[mythtv] Convoluted program expiry and deletion

Ian Dall ian at beware.dropbear.id.au
Thu May 10 22:56:31 UTC 2012


On Thu, 2012-05-10 at 10:37 -0400, Michael T. Dean wrote:
> On 05/10/2012 09:55 AM, Ian Dall wrote:
> > I have been trying to debug the program deletion code (Ticket #10704).
> > However, this post is really about how convoluted the program deletion
> > actually has become and some suggestions for small improvement.
> >
> > As I was debugging I took some notes about how the process works (or
> > doesn't). It looks to me like the "Change delete behaviour so that we
> > always use the deleted recording group" commit
> > f78f9992d754390fa42f109e5139b8eaf224d076, has left some artefacts which
> > may have made sense once, but now seem pointless. The whole process
> > seems excessively complicated.
> 
> We appreciate the work you've done on tracking the existing code, but I 
> thought I'd just send a quick e-mail to give you a bit of info on why 
> we're where we are.
> 
> [..]

Noted.

> The plan is to simplify the code, but we didn't have time to do so (let 
> alone time for fully testing a complex change) before 0.25 release.

I am happy to take this on if it doesn't step on anyone's toes. I think
understanding the existing code was the hardest bit. After that the
places to clean up seem relatively obvious.

Regards,
Ian

-- 
Ian Dall <ian at beware.dropbear.id.au>



More information about the mythtv-dev mailing list