[FFmpeg-devel] [patch] export pmt and pcr pids from mpegts demuxer

[Michael Niedermayer]

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 
>>> copied
>>> blindly.
>>> The problem is that pcr_pid&  pmt_pid is not data describing the 
>>> video/audio
>>> 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 ?
> Michael,
>
> 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 
> acceptable.

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
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100620/80756db6/attachment.pgp>