[FFmpeg-devel] [patch] export pmt and pcr pids from mpegts demuxer
Sun Jun 20 00:10:04 CEST 2010
On Sat, Jun 19, 2010 at 09:35:41AM -0400, Mike Scheutzow wrote:
> Baptiste Coudurier wrote:
>> On 06/10/2010 03:45 PM, Michael Niedermayer wrote:
>>> metadata (that is data describing the video and audio content) can be
>>> The problem is that pcr_pid& pmt_pid is not data describing the
>>> its data describing the mpeg stream
>> It's the same as major brand from mp4.
>> We are turning around here.
>> Polluting AVStream/AVFormatContext with this format specific information
>> seems ugly to me. We are not going to add a field for every value we want
>> to export.
>> So what do you propose ? Another field which is not called metadata ?
> I agree with Baptiste that adding very format-specific fields to AVProgram
> or AVStream makes the API more difficult to understand.
> Please give me a hint about an implementation that you would find
Please elaborate on how and for what they would be used:
"iam not saying to put them in AVStream, actually iam not sure myself where
to put them, maybe you can elaborate a bit on the use cases of these fields?
Extending AVOptions to allow accessing demuxer specific contexts might be
an option maybe but then maybe not iam not sure."
I cant make a suggestion if i dont know exactly what these fields would be
used for. I can read the mpeg spec but that will take time
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
If a bugfix only changes things apparently unrelated to the bug with no
further explanation, that is a good sign that the bugfix is wrong.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 189 bytes
Desc: Digital signature
More information about the ffmpeg-devel