[FFmpeg-devel] PATCH: mpegtsenc.c

[Georgi Chorbadzhiyski]

On 1/19/11 3:47 PM, Tomas H?rdin wrote:
> Georgi Chorbadzhiyski skrev 2011-01-19 08:20:
>> Around 01/19/11 04:02, M?ns Rullg?rd scribbled:
>>> JULIAN GARDNER<joolzg at btinternet.com> writes:
>>>
>>>> Hi new user here.
>>>>
>>>> I have the following patch which allows the user through the
>>>> --metadata to also change the provider name on TS files, a bit like
>>>> the title option changes the service name.
>>
>> Two old patches of mine that allow
>>
>> ffmpeg -metadata provider_name="Some provider name" -metadata
>> service_name="Some TV channel"
>>
>> to work for mpeg ts files/streams.
>
> Seems like a decent solution, however: shouldn't AVOption be used for
> this? This would allow listing such things in the ffmpeg CLI. It would
> also be good for other things, like settings the start PID and the PMT
> start PID. I have no strong feelings about it though.

It certainly will be better with AVOption (the help texts and all) but
is it possible to use AVOption in muxers/demuxers? I briefly looked in
lavf usage of AVOptions and only found this:

avio.c:static const AVOption options[] = {{NULL}};
http.c:static const AVOption options[] = {
options.c:static const AVOption options[]={

 From the quick look I can't see how to get AVOption just in mpegtsenc.c
I really don't want to pollute AVFormatContext.

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