On 2011-12-21 11:02:33 +0100, Benjamin Larsson wrote:
> On 12/20/2011 05:43 PM, Janne Grunau wrote:
> >From: Tomas Härdin<[email protected]>
> >
> >---
> >  libavformat/mxfdec.c |  108 
> > +++++++++++++++++++++++++++++++++++++++++++++++++-
> >  1 files changed, 107 insertions(+), 1 deletions(-)
> >
> >diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c
> >index 2a455d7..7d153dc 100644
> >--- a/libavformat/mxfdec.c
> >+++ b/libavformat/mxfdec.c
> >@@ -52,6 +52,35 @@
> >  #include "internal.h"
> >  #include "mxf.h"
> >
> >+typedef enum {
> >+    Header,
> >+    BodyPartition,
> >+    Footer
> >+} MXFPartitionType;
> >+
> >+typedef enum {
> >+    OP1a,
> >+    OP1b,
> >+    OP1c,
> >+    OP2a,
> >+    OP2b,
> >+    OP2c,
> >+    OP3a,
> >+    OP3b,
> >+    OP3c,
> >+    OPAtom,
> >+} MXFOP;
> >+
> >
> 
> I like my enums and defines to have capital letters.

I can change them but I'm not sure if it's worth it.

Do it as a seperate clean-up patch at the end of the series might be
not to painful.

Janne
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to