[FFmpeg-devel] PATCH: mpegtsenc.c

[Peter Ross]

On Thu, Jan 20, 2011 at 08:47:04AM +0100, Anton Khirnov wrote:
> > From 1ef479c6ff440c0f02147b1562895cafd24b34ad Mon Sep 17 00:00:00 2001
> > From: Georgi Chorbadzhiyski <gf at unixsol.org>
> > Date: Thu, 20 Jan 2011 09:26:40 +0200
> > Subject: [PATCH] Allow provider name and service name to be set in mpegts muxer
> > 
> > With this patch mpegts muxer uses provider_name and service_name (in
> > addition to title) metadata to set service_provider_name and service_name
> > fields in SDT.
> > 
> > Example usage:
> >    ffmpeg -i file.ts -f mpegts -acodec copy -vcodec copy -f mpegts -metadata provider_name="Some provider" -metadata service_name="Some Channel" "udp://239.0.70.2:5000?pkt_size=1316&ttl=1"
> > ---
> >  libavformat/avformat.h  |    2 ++
> >  libavformat/mpegtsenc.c |   10 +++++++---
> >  2 files changed, 9 insertions(+), 3 deletions(-)
> > 
> > diff --git a/libavformat/avformat.h b/libavformat/avformat.h
> > index b484649..1f4e0bc 100644
> > --- a/libavformat/avformat.h
> > +++ b/libavformat/avformat.h
> > @@ -163,6 +163,8 @@ struct AVFormatContext;
> >   * publisher    -- name of the label/publisher.
> >   * title        -- name of the work.
> >   * track        -- number of this work in the set, can be in form current/total.
> > + * provider_name-- Set service_provider_name when using mpegts muxer.
> > + * service_name -- Set service_name when using mpegts muxer.
> I don't like it explicitly referencing the mpegts muxer, how about

Ditto. The WTV demuxer can populate these fields also.

-- Peter
(A907 E02F A6E5 0CD2 34CD 20D2 6760 79C5 AC40 DD6B)
-------------- 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/20110120/b8399b76/attachment.pgp>