[mythtv] Re: [patch]building a tree for video{tree|gallery}

Leo Weppelman leo at wau.mis.ah.nl
Tue Sep 28 02:42:32 EDT 2004


This is a resend with a gzipped patch. The original seems to be hanging
on moderator approval because of the message body size...

On Sun, Sep 26, 2004 at 10:42:22PM +0200, Leo Weppelman wrote:
> This patch tries to pull-out the tree building code out of the
> various views/classes into a class of it's own. 
> 
> It is my first major c++ project, so beware ;-) Thor was willing to preview
> it, but he didn't have the time to actually do it. So we decided to throw it
> on this list.
> 
> There is one 'bug' in the code that I know of, but have no fix for yet:
>    In the videotree, you now see the 'updir's popping up. They have no
>    in this view. But I couldn't find an easy way to suppress them. An
>    opion would be to pass a flag to videolist::buildVideoList() to
>    not include them. But that would be harmfull to the next item...
> 
> Another thing that is missing is actually an idea from Thor, that it would
> be nice to be able to pass the tree around between the various views without
> the need to rebuild it. I haven't implemented it, because I could not find
> a good way to do it and because the current patch is already pretty big.
> 
> PS: After attaching the diff's I saw that videolist.cpp still had 3 debug
>    'cerr' statements left. They all start with cerr at position 0...
> 
> Leo.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: diff-videolist-20040926.gz
Type: application/x-gunzip
Size: 8729 bytes
Desc: not available
Url : http://mythtv.org/pipermail/mythtv-dev/attachments/20040928/279ebecf/diff-videolist-20040926.bin


More information about the mythtv-dev mailing list