[mythtv] DVB Code Update

Ben Bucksch linux.news at bucksch.org
Wed Aug 27 09:09:41 EDT 2003


Ben Bucksch wrote:

> That's a lot better than the earlier versions. A few comments (I 
> haven't read through all the code):

I just saw that you unnecessarily dropped support for use_ts = 1 (filter 
PIDs in MythTV). It may have had a bug, but it's still an important mode 
for later when we support multiple recordings per card, and it was a 
design goal when I wrote the code. Current cards/drivers can only filter 
a limited number of PIDs, which is likely not to be sufficient for 
multiple recordings.

If you knowingly break features in your patches, you should warn about 
that clearly.


You dragged ts_to_ps into dvbrecorder.cpp. If you copy code from 
elsewhere (even when with changes), please add a comment (right above 
the copied/adapted code) where it comes from. It gives credit and helps 
keeping the code in sync in case of bugs.



More information about the mythtv-dev mailing list