[linux-dvb] Re: It works!

[Marcus Metzler <mocm@metzlerbros.de>]

Robert Schlabbach writes:
 > From: "Johannes Stezenbach" <js@convergence.de>
 > > Robert Schlabbach wrote:
 > > > From: "Holger Waechtler" <holger@convergence.de>
 > > > > packet_length can be 130 or 188bytes, valid values for packet_stride
 > > > > could be 130, 188 and 204 bytes. If all constants '188' get replaced
 > > > > by these variables this should be flexible enough for all cases.
 > > >
 > > > Umm, what about 147 bytes? If I get this right, DSS uses 147 bytes
 > > > packets, of which 130 bytes are data, correct? So shouldn't a stride
 > > > of 147 bytes be supported as well...?
 > >
 > > Who cares about DSS?
 > >
 > > Rule #23: Don't implement stuff that maybe someone might need sometime.
 > > Wait until at least one user shows up who actually needs it.
 > 
 > And why do you support 130 bytes packet length then? It would make sense to
 > support neither or both 130 and 147 bytes, but IMHO it doesn't make sense
 > to have "half" DSS support in there.
 > 

I would say that we only support MPEG transport packets. You could
make an exception for packets with a different length and the same
kind of syntax (i.e. same headers, using sections and adaptation
fields ), but to support a totally different type of container format
which only happens to have the same payload doesn't make sense. If you
stay with the TS syntax and only change the length it should be
possible to handle different sizes, by just changing the TS_SIZE
definition (barring cases where writing 186 was easier to write than
TS_SIZE-2) and make it a variable.
I guess for DSS such things as PID and section filters don't make any
sense anyway and would require a different API.

Marcus

PS: The only thing I found about the DSS format on the net is
    http://www.coolstf.com/mpeg/#dss

-- 
/--------------------------------------------------------------------\
| Dr. Marcus O.C. Metzler        |                                   |
| mocm@metzlerbros.de            | http://www.metzlerbros.de/        |
\--------------------------------------------------------------------/
 |>>> This e-mail was encrypted using the ROT13 algorithm twice. <<<|




-- 
Info:
To unsubscribe send a mail to ecartis@linuxtv.org with "unsubscribe linux-dvb" as subject.