[mythtv] [PATCH] mythvideo cd support
maxpower44 at tiscali.fr
Thu Jul 1 09:02:23 EDT 2004
Xavier Hervy wrote:
> Hi Matt,
> thx to try it
> Matt Porter wrote:
>> On Wed, Jun 30, 2004 at 06:26:17PM +0200, Xavier Hervy wrote:
>>> My patch should be clean now, sorry for my dirty patch ;-)
>> Much better. After I got the first one building, it didn't
>> properly launch the media handler...then the new patch appeared.
>> There's still a couple warnings
> Warning is normal because i use virtual function that used some
> parameter in parent class but not in child
>> and a few debugging couts left,
> I believe that i remove all fo them ?
>> I didn't have time to do much debugging (had to rebuild for
>> debug and go to work), but the VideoBrowser segfaults in several
>> scenarios when manipulating removable media. I'm successfully
>> able to get it to autoadd and then select a movie from the VideoBrowser
>> to play. However, if I eject the media, then select the movie it
>> gives me the dialog to reinsert "CDROM". At this point, whether I
>> select Retry or Cancel or have the media inserted/removed causes
>> a segfault. videotree works fine like the original patch...haven't
>> tried ejecting there. I'll debug when I get home if somebody hasn't
>> already fixed the issue.
> Strange. did you try to do the same thing in videogalery or in videotree
> (with the option "show new file" disabled) ?
>>> TODO :
>>> -optimize getNumVideo in VideoFilterSettings i.e. execute query only if
>>> on criteria have change otherwise return the old store number
>>> - optimize display in videogallery
>>> - integrate parentallevel in videofiltersetting class then the number of
>>> video display in filterdialog and in videobrowser will be the same.
>> One thing I'd like to add once it's stable is the assignment of a human
>> readable ID tag so if I select something from the browser it will ask
>> me to insert the media tagged "FOO". It would make it easy to find
>> the correct optical disk in say a folder of a hundred.
> Yes, i would like to do something like that. probalbly when you insert
> the first time the media, and allow to modify this tag in editmetadata.
> somebody tell me that they used "book" to store cd like me then i was
> thinking to add an entry in video setting wich is a pattern. then for me
> i can put "Page %1" then when you insert a media you could select Page
> 1, page 2, .... Page n, or enter a name yourself.
> I will retry videobrowser to find what is wrong.
I have tested my patch on a fresh cvs version.
I don't have the bug you talk about.
but frontend crash if i go in videobrowser -> filter display
then i press "Done" or "save as default" button and frontend crash
I fix it and then resubmit patch.
About warning, for one of them i made a stupid test on a parameter to
remove the warning (if (modetree != modetree) return "";)
For the two others, i made something now with parameter then they should
Finally, in videogallery, when you choose to group by storage, the query
return no results. i fix it.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 24750 bytes
Desc: not available
Url : http://mythtv.org/pipermail/mythtv-dev/attachments/20040701/41c11e55/mythvideocdsupport.tar.obj
More information about the mythtv-dev