[FFmpeg-devel] PATCH: mpegtsenc.c

[Georgi Chorbadzhiyski]

Around 01/20/11 09:47, Anton Khirnov scribbled:
>> 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
> provider_name -- name of the service provider in broadcasting
> (and maybe the '_name' is redundant, service_provider looks better to me)
> Also please keep the list alphabetically sorted.
>>   */

Please see the attached file. It contains the proposed changes.

-- 
Georgi Chorbadzhiyski
http://georgi.unixsol.org/