From ae1f22242a06390342295a0bda5c7d37c90be7fe Mon Sep 17 00:00:00 2001
From: Steve Lhomme <slhomme@matroska.org>
Date: Sat, 6 Jan 2024 10:36:00 +0100
Subject: [PATCH] add a macro to define the body of a Matroska Class

So we can extend it later if needed.

It's also clearer what the macro does.
---
 matroska/KaxBlock.h   |  4 ++--
 matroska/KaxDefines.h | 35 +++++++++++++++++++----------------
 2 files changed, 21 insertions(+), 18 deletions(-)

diff --git a/matroska/KaxBlock.h b/matroska/KaxBlock.h
index c7f536b7..ef3d441a 100644
--- a/matroska/KaxBlock.h
+++ b/matroska/KaxBlock.h
@@ -270,7 +270,7 @@ class MATROSKA_DLL_API KaxBlock : public KaxInternalBlock {
     static const libebml::EbmlCallbacks ClassInfos;
   public:
     KaxBlock() :KaxInternalBlock(KaxBlock::ClassInfos, false) {}
-        EBML_CONCRETE_CLASS(KaxBlock)
+    MATROSKA_CLASS_BODY(KaxBlock)
 };
 
 class MATROSKA_DLL_API KaxSimpleBlock : public KaxInternalBlock {
@@ -287,7 +287,7 @@ class MATROSKA_DLL_API KaxSimpleBlock : public KaxInternalBlock {
 
     void SetParent(KaxCluster & aParentCluster);
 
-        EBML_CONCRETE_CLASS(KaxSimpleBlock)
+    MATROSKA_CLASS_BODY(KaxSimpleBlock)
 };
 
 /// Placeholder class for either a BlockGroup or a SimpleBlock
diff --git a/matroska/KaxDefines.h b/matroska/KaxDefines.h
index 69369f70..693991b9 100644
--- a/matroska/KaxDefines.h
+++ b/matroska/KaxDefines.h
@@ -31,84 +31,87 @@
 #define DEFINE_MKX_SINTEGER_CONS(a,b,c,d,e)  DEFINE_xxx_CLASS_CONS(a,b,c,d,e,GetKaxGlobal_Context) \
     const EbmlCallbacksDefault<std::int64_t> a::ClassInfos(a::Create, Id_##a, e, Context_##a);
 
+#define MATROSKA_CLASS_BODY(x) \
+    EBML_CONCRETE_CLASS(x)
+
 #define DECLARE_MKX_MASTER(x)   \
     DECLARE_xxx_MASTER(x, MATROSKA_DLL_API) \
     x(const x & ElementToClone) :EbmlMaster(ElementToClone) {} \
-    EBML_CONCRETE_CLASS(x)
+    MATROSKA_CLASS_BODY(x)
 
 #define DECLARE_MKX_MASTER_CONS(x)   \
     DECLARE_xxx_MASTER(x, MATROSKA_DLL_API) \
     x(const x & ElementToClone); \
-    EBML_CONCRETE_CLASS(x)
+    MATROSKA_CLASS_BODY(x)
 
 #define DECLARE_MKX_BINARY(x)   \
     DECLARE_xxx_BINARY(x, MATROSKA_DLL_API) \
     x(const x & ElementToClone) :libebml::EbmlBinary(ElementToClone) {} \
-    EBML_CONCRETE_CLASS(x)
+    MATROSKA_CLASS_BODY(x)
 
 #define DECLARE_MKX_BINARY_CONS(x)   \
     DECLARE_xxx_BINARY(x, MATROSKA_DLL_API) \
     x(const x & ElementToClone); \
-    EBML_CONCRETE_CLASS(x)
+    MATROSKA_CLASS_BODY(x)
 
 #define DECLARE_MKX_UNISTRING(x) \
     DECLARE_xxx_UNISTRING(x, MATROSKA_DLL_API) \
     x(const x & ElementToClone) :libebml::EbmlUnicodeString(ElementToClone) {} \
-    EBML_CONCRETE_CLASS(x)
+    MATROSKA_CLASS_BODY(x)
 
 #define DECLARE_MKX_STRING(x)   \
     DECLARE_xxx_STRING(x, MATROSKA_DLL_API) \
     x(const x & ElementToClone) :libebml::EbmlString(ElementToClone) {} \
-    EBML_CONCRETE_CLASS(x)
+    MATROSKA_CLASS_BODY(x)
 
 #define DECLARE_MKX_STRING_DEF(x)   \
     DECLARE_xxx_STRING_DEF(x, MATROSKA_DLL_API) \
     x(const x & ElementToClone) :libebml::EbmlString(ElementToClone) {} \
-    EBML_CONCRETE_CLASS(x)
+    MATROSKA_CLASS_BODY(x)
 
 #define DECLARE_MKX_UINTEGER(x) \
     DECLARE_xxx_UINTEGER(x, MATROSKA_DLL_API) \
     x(const x & ElementToClone) :libebml::EbmlUInteger(ElementToClone) {} \
-    EBML_CONCRETE_CLASS(x)
+    MATROSKA_CLASS_BODY(x)
 
 #define DECLARE_MKX_UINTEGER_DEF(x) \
     DECLARE_xxx_UINTEGER_DEF(x, MATROSKA_DLL_API) \
     x(const x & ElementToClone) :libebml::EbmlUInteger(ElementToClone) {} \
-    EBML_CONCRETE_CLASS(x)
+    MATROSKA_CLASS_BODY(x)
 
 #define DECLARE_MKX_SINTEGER_CONS(x) \
     DECLARE_xxx_SINTEGER(x, MATROSKA_DLL_API) \
     x(const x & ElementToClone); \
-    EBML_CONCRETE_CLASS(x)
+    MATROSKA_CLASS_BODY(x)
 
 #define DECLARE_MKX_SINTEGER(x) \
     DECLARE_xxx_SINTEGER(x, MATROSKA_DLL_API) \
     x(const x & ElementToClone) :libebml::EbmlSInteger(ElementToClone) {} \
-    EBML_CONCRETE_CLASS(x)
+    MATROSKA_CLASS_BODY(x)
 
 #define DECLARE_MKX_SINTEGER_DEF(x) \
     DECLARE_xxx_SINTEGER_DEF(x, MATROSKA_DLL_API) \
     x(const x & ElementToClone) :libebml::EbmlSInteger(ElementToClone) {} \
-    EBML_CONCRETE_CLASS(x)
+    MATROSKA_CLASS_BODY(x)
 
 #define DECLARE_MKX_DATE(x)     \
     DECLARE_xxx_DATE(x, MATROSKA_DLL_API) \
     x(const x & ElementToClone) :libebml::EbmlDate(ElementToClone) {} \
-    EBML_CONCRETE_CLASS(x)
+    MATROSKA_CLASS_BODY(x)
 
 #define DECLARE_MKX_DATE_DEF(x)     \
     DECLARE_xxx_DATE_DEF(x, MATROSKA_DLL_API) \
     x(const x & ElementToClone) :libebml::EbmlDate(ElementToClone) {} \
-    EBML_CONCRETE_CLASS(x)
+    MATROSKA_CLASS_BODY(x)
 
 #define DECLARE_MKX_FLOAT(x)    \
     DECLARE_xxx_FLOAT(x, MATROSKA_DLL_API) \
     x(const x & ElementToClone) :libebml::EbmlFloat(ElementToClone) {} \
-    EBML_CONCRETE_CLASS(x)
+    MATROSKA_CLASS_BODY(x)
 
 #define DECLARE_MKX_FLOAT_DEF(x)    \
     DECLARE_xxx_FLOAT_DEF(x, MATROSKA_DLL_API) \
     x(const x & ElementToClone) :libebml::EbmlFloat(ElementToClone) {} \
-    EBML_CONCRETE_CLASS(x)
+    MATROSKA_CLASS_BODY(x)
 
 #endif // LIBMATROSKA_DEFINES_H