[mythtv] [PATCH] DVB support

Ben Bucksch linux.news at bucksch.org
Fri May 2 19:25:01 EDT 2003


Hi Isaac, thanks for the quick response.

Isaac wrote:

>I asked you not to munge up MpegRecorder at all, but you did.  Inherit off of 
>the base recorder class.
>
I didn't understand you that way. Should I just *copy* all your code 
which is in mpegrecorder after my patch (for seeking etc.), 320 lines in 
24 functions, although it's identical in 3 files, which all only have 
150 lines and 6 functions or less right now?

>I'd also actually prefer it if you kept channel.cpp/h as the v4l interface, created a basechannel.cpp/h base class, and then inherited your dvb channel stuff off of that.
>
That means that I have to rename *all* the references to |Channel| in 
all other files. Also, I find it confusing to have the analog-specific 
part having a generic name like Channel.

>And how are people supposed to use the new format?  Manual db interaction?  
>You didn't update filldatabase or anything to go along with it.
>
I thought I'd wait for your reaction before I write code that you may 
eventually reject anyways. Or ask me to use another way to store the 
parameters.

>What's with the chan->GetCardID() -- isn't that already a member variable in 
>the TVRec class?
>
Possible, havn't noticed that, will check.

>Everything else looks ok, from a quick once-over.
>
That's great. Before I submit a new patch (dissecting the patches 
manually costs time), could you please tell me any other complaints you 
may have, so that I can fix them in the next batch of patches, having 
them ready to be checked in?

>I'm not planning on applying anything until after the 0.9 release, anyway
>
Too bad. I was hoping to get that puppy in in the next days, partially 
to avoid conflicts and having to fix them manually.



More information about the mythtv-dev mailing list