[FFmpeg-devel] [PATCH] forward ASF tags to RTSP for MS-RTSP

Anton Khirnov anton
Wed Jan 12 21:41:56 CET 2011


On Wed, Jan 12, 2011 at 02:07:11PM +0100, Aurelien Jacobs wrote:
> On Tue, Jan 11, 2011 at 10:39:37PM -0500, Ronald S. Bultje wrote:
> > Hi,
> > 
> > On Mon, Jan 3, 2011 at 6:48 PM, Aurelien Jacobs <aurel at gnuage.org> wrote:
> > > On Mon, Jan 03, 2011 at 03:06:20PM -0500, Justin Ruggles wrote:
> > >> On 01/03/2011 01:22 PM, Ronald S. Bultje wrote:
> > >>
> > >> > Hi,
> > >> >
> > >> > On Mon, Jan 3, 2011 at 1:17 PM, Martin Storsj? <martin at martin.st> wrote:
> > >> >> On Mon, 3 Jan 2011, Ronald S. Bultje wrote:
> > >> >>
> > >> >>> for chained demuxers, we need to manually FW tags, this patch does it
> > >> >>> for MS-RTSP. Fixes issue 2478.
> > >> >>
> > >> >> Looks good to me.
> > >> >>
> > >> >> As future improvment - should we make this into a reusable function? I
> > >> >> guess something similar might be needed for other
> > >> >> full-container-within-another-container setups (like mpegts in RTP).
> > >> >
> > >> > Sounds like a good plan - but I'd like to leave that question to other
> > >> > metadata experts. Anton? Aurel? Baptiste? Michael?
> > >>
> > >>
> > >> search ffmpeg.c for "autocopy" and you'll find lots of code very similar
> > >> to this. ?So adding it to the public API might be a good idea.
> > >
> > > I agree that a public metadata_copy API would be useful.
> > > Something like:
> > > ? ?int av_metadata_copy(AVMetadata **dest, AVMetadata *src);
> > 
> > Has anyone implemented this since then?
> 
> Not that I know of, but IIRC there was some patches sent here a long
> time ago proposing something similar (and never accepted).
> 
> > Or should I do it myself?
> 
> This shouldn't be hard to do, so I would say yes, go ahead.
I submitted a rather cumbersome copy&convert function; then it was
decided it'd be simpler/better to just drop metadata conversion from
public API so the patch was abandoned.

Anyway I'm kinda busy with exams right now, so don't wait for me ;)

-- 
Anton Khirnov
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20110112/2cf6bbab/attachment.pgp>



More information about the ffmpeg-devel mailing list