On Wed, Nov 29, 2017 at 12:56 PM, Matthias Troffaes <
matthias.troff...@gmail.com> wrote:
> Dear Mark,
>
> On Wed, Nov 29, 2017 at 4:11 AM, Mark Reid <mindm...@gmail.com> wrote:
> > @@ -1445,12 +1463,13 @@ static int
> > mxf_write_header_metadata_sets(AVFormatContext
> *s)
> > AVDictionaryEntry *entry = NULL;
> > AVStream *st = NULL;
> > int i;
> > -
> > - MXFPackage packages[2] = {};
> > + MXFPackage packages[3] = {};
>
> Here, may I suggest
>
> MXFPackage packages[3] = {0};
>
> for C99 compliance? For instance, msvc does not support an empty
> struct initializer. See for instance
> https://stackoverflow.com/questions/17589533/is-an-
> empty-initializer-list-valid-c-code
>
> Kind regards,
> Matthias
>
sorry about that! yes I'll send a patch fixing that.
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
