Re: [FFmpeg-devel] [PATCH 1/3] avformat/mxfenc: Replace more literal magic numbers by enum values.

2017-09-12 Thread Michael Niedermayer
On Sun, Sep 10, 2017 at 10:28:05PM +0200, Tomas Härdin wrote: > On Sun, 2017-09-10 at 22:10 +0200, Michael Niedermayer wrote: > >  enum { > >  INDEX_MPEG2 = 0, > >  INDEX_AES3, > > @@ -159,6 +139,26 @@ enum { > >  INDEX_H264, > >  }; > >   > > +static const struct { > > +enum

Re: [FFmpeg-devel] [PATCH 1/3] avformat/mxfenc: Replace more literal magic numbers by enum values.

2017-09-12 Thread Michael Niedermayer
On Sun, Sep 10, 2017 at 10:28:05PM +0200, Tomas Härdin wrote: > On Sun, 2017-09-10 at 22:10 +0200, Michael Niedermayer wrote: > >  enum { > >  INDEX_MPEG2 = 0, > >  INDEX_AES3, > > @@ -159,6 +139,26 @@ enum { > >  INDEX_H264, > >  }; > >   > > +static const struct { > > +enum

Re: [FFmpeg-devel] [PATCH 1/3] avformat/mxfenc: Replace more literal magic numbers by enum values.

2017-09-10 Thread Tomas Härdin
On Sun, 2017-09-10 at 22:10 +0200, Michael Niedermayer wrote: >  enum { >  INDEX_MPEG2 = 0, >  INDEX_AES3, > @@ -159,6 +139,26 @@ enum { >  INDEX_H264, >  }; >   > +static const struct { > +enum AVCodecID id; > +int index; > +} mxf_essence_mappings[] = { > +{

[FFmpeg-devel] [PATCH 1/3] avformat/mxfenc: Replace more literal magic numbers by enum values.

2017-09-10 Thread Michael Niedermayer
This also moves the enum table up as it is needed earlier Signed-off-by: Michael Niedermayer --- libavformat/mxfenc.c | 40 1 file changed, 20 insertions(+), 20 deletions(-) diff --git a/libavformat/mxfenc.c