[FFmpeg-devel] [PATCH 2/5] copy stream metadata when using concat

[Vadim Belov]

Thanks Nicolas,
I reconfigured Git and sent again.
It was tested and the added functionality works.


On Wed, Jul 1, 2015 at 1:19 PM, Nicolas George <george at nsup.org> wrote:

> Le tridi 13 messidor, an CCXXIII, Vadim Belov a écrit :
> > From: unknown <vadimb at VADIMB-T440.nice.com>
>
> Not good. Please configure Git to include a valid name and address.
>
> >
> > ---
> >  libavformat/concatdec.c | 2 ++
> >  1 file changed, 2 insertions(+)
> >
> > diff --git a/libavformat/concatdec.c b/libavformat/concatdec.c
> > index f07cfd7..c61d5c2 100644
> > --- a/libavformat/concatdec.c
> > +++ b/libavformat/concatdec.c
> > @@ -172,6 +172,8 @@ static int copy_stream_props(AVStream *st, AVStream
> *source_st)
> >      st->avg_frame_rate      = source_st->avg_frame_rate;
> >      st->time_base           = source_st->time_base;
> >      st->sample_aspect_ratio = source_st->sample_aspect_ratio;
> > +
> > +    av_dict_copy(&st->metadata, source_st->metadata, 0);
> >      return 0;
> >  }
>
> Code change LGTM if tested.
>
> (I am a bit shocked that av_dict_copy() does not return an error code, but
> that is not your fault.)
>
> Regards,
>
> --
>   Nicolas George
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
>