On 21/12/11 11:43, Janne Grunau wrote:
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<tomas.har...@codemill.se>

---
  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.

sed -i -e on the mailbox works wonders =)

lu

--

Luca Barbato
Gentoo/linux
http://dev.gentoo.org/~lu_zero

_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to