[FFmpeg-devel] PATCH: mpegtsenc.c
Wed Jan 19 22:31:42 CET 2011
Georgi Chorbadzhiyski <gf at unixsol.org> writes:
> On 1/19/11 2:49 PM, M?ns Rullg?rd wrote:
>> Georgi Chorbadzhiyski<gf at unixsol.org> writes:
>>> 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.
>> Code looks good, but perhaps the _name suffix on the tags could be dropped.
>> No strong opinion there, just thought it seemed redundant.
> The fields are named after the names used in the relevant DVB standard
> (EN 300 468, 6.2.31 Service descriptor). Actually it should be
> "service_provider_name" but that is really excessive. I really don't
> care about the naming as long as I can set the service provider name :)
Makes sense. I'll leave metadata-specific review to Anton. He's the
man in that area.
mans at mansr.com
More information about the ffmpeg-devel