[FFmpeg-devel] PATCH: mpegtsenc.c

[Måns Rullgård]

JULIAN GARDNER <joolzg at btinternet.com> writes:

> --- On Thu, 20/1/11, Georgi Chorbadzhiyski <gf at unixsol.org> wrote:
>
>> Subject: [PATCH 2/2] Set service_provider and service_name
>> in mpegts demuxer
> ...
>> 
>> Set service_provider and service_name in mpegts demuxer,
>> previously
>> name and provider_name were set but since the muxer uses
>> service_provider
>> and service_name use them.
>> ---
>>  libavformat/mpegts.c |? ? 4 ++--
>>  1 files changed, 2 insertions(+), 2 deletions(-)
>> 
>> diff --git a/libavformat/mpegts.c b/libavformat/mpegts.c
>> index 01b69a7..d234316 100644
>> --- a/libavformat/mpegts.c
>> +++ b/libavformat/mpegts.c
>> @@ -1211,8 +1211,8 @@ static void sdt_cb(MpegTSFilter
>> *filter, const uint8_t *section, int section_len
>> ? ? ? ? ? ? ?
>> ???if (name) {
>> ? ? ? ? ? ? ? ?
>> ? ???AVProgram *program =
>> av_new_program(ts->stream, sid);
>> ? ? ? ? ? ? ? ?
>> ? ???if(program) {
>> -? ? ? ? ? ? ? ?
>> ? ? ? ?
>> av_metadata_set2(&program->metadata, "name", name,
>> 0);
>> -? ? ? ? ? ? ? ?
>> ? ? ? ?
>> av_metadata_set2(&program->metadata, "provider_name",
>> provider_name, 0);
>> +? ? ? ? ? ? ? ?
>> ? ? ? ?
>> av_metadata_set2(&program->metadata, "service_name",
>> name, 0);
>> +? ? ? ? ? ? ? ?
>> ? ? ? ?
>> av_metadata_set2(&program->metadata,
>> "serivce_provider", provider_name, 0);
>
> Should this not be "service_provider"

Obviously.  Queued with this change.

-- 
M?ns Rullg?rd
mans at mansr.com