[FFmpeg-devel] [PATCH] RTSP muxer, round 2

[Martin Storsjö]

On Mon, 11 Jan 2010, Michael Niedermayer wrote:

> > - Are there any other stream parameters that need to be copied to the
> >   AVStream objects of the chained muxers, other than sample_aspect_ratio?
> > - Which stream parameters need to be copied back to the original AVStream
> >   from the opened chained AVStream? Currently time_base is copied.
> 
> whatever it is it belongs into a function in utils.c

Would a change like the attached one be ok? (The patch is against the 
combined result of the patch series, but shows the new utility functions.)

For initializing the AVCodecContext, I'd say its best to leave it up to 
each occasion of chained muxers - in this case simply referring to the 
codec context of the master is the easiest, but in some other cases one 
may want to copy most of it but change some fields. In that case, we'd 
probably need some kind of avcodec_context_copy(), to copy/initialize the 
fields correctly.

// Martin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: chained-muxer-utils.patch
Type: text/x-diff
Size: 2399 bytes
Desc: 
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100112/bcdfa69f/attachment.patch>