always compile in strict API mode

All the members become private.

This requires libmatroska to merge Matroska-Org/libmatroska#95

Author: robUx4

ebml/EbmlBinary.h

@@ -88,11 +88,7 @@ class EBML_DLL_API EbmlBinary : public EbmlElement {
 
     bool operator==(const EbmlBinary & ElementToCompare) const;
 
-#if defined(EBML_STRICT_API)
   private:
-#else
-  protected:
-#endif
     binary *Data{nullptr}; // the binary data inside the element
 };
 

ebml/EbmlCrc32.h

@@ -85,11 +85,7 @@ DECLARE_EBML_BINARY(EbmlCrc32)
 
     void ForceCrc32(std::uint32_t NewValue) { m_crc_final = NewValue; SetValueIsSet();}
 
-#if defined(EBML_STRICT_API)
     private:
-#else
-    protected:
-#endif
     void ResetCRC();
     void UpdateByte(binary b);
 

ebml/EbmlDate.h

@@ -82,11 +82,7 @@ class EBML_DLL_API EbmlDate : public EbmlElement {
       return false;
     }
 
-#if defined(EBML_STRICT_API)
     private:
-#else
-    protected:
-#endif
     filepos_t RenderData(IOCallback & output, bool bForceRender, bool bWithDefault = false) override;
 
     std::int64_t myDate{0}; ///< internal format of the date

ebml/EbmlDummy.h

@@ -52,11 +52,7 @@ class EBML_DLL_API EbmlDummy : public EbmlBinary {
             return DummyId;
         }
 
-#if defined(EBML_STRICT_API)
     private:
-#else
-    protected:
-#endif
     const EbmlId DummyId;
     static const EbmlId DummyRawId;
 

ebml/EbmlElement.h

@@ -176,7 +176,6 @@ extern const EbmlSemanticContext Context_EbmlGlobal;
   public: \
     x();
 
-#if defined(EBML_STRICT_API)
 #define EBML_CONCRETE_CLASS(Type) \
     public: \
         const EbmlSemanticContext &Context() const override {return ClassInfos.GetContext();} \
@@ -225,50 +224,6 @@ extern const EbmlSemanticContext Context_EbmlGlobal;
 #define EBML_CTX_IDX(c,i)      (c).GetSemantic(i)
 #define EBML_CTX_IDX_INFO(c,i) (const EbmlCallbacks &)((c).GetSemantic(i))
 #define EBML_CTX_IDX_ID(c,i)   ((const EbmlCallbacks &)((c).GetSemantic(i))).ClassId()
-#else
-#define EBML_CONCRETE_CLASS(Type) \
-    public: \
-    const EbmlCallbacks & Generic() const override {return ClassInfos;} \
-    operator const EbmlId &() const override {return ClassInfos.GlobalId;} \
-        EbmlElement & CreateElement() const override {return Create();} \
-        EbmlElement * Clone() const override { return new Type(*this); } \
-    static EbmlElement & Create() {return *(new (Type));} \
-    static const EbmlCallbacks ClassInfos; \
-
-#define EBML_CONCRETE_DUMMY_CLASS(Type) \
-    public: \
-    const EbmlCallbacks & Generic() const override {return ClassInfos;} \
-    virtual operator const EbmlId &(); \
-        EbmlElement & CreateElement() const override {return Create();} \
-        EbmlElement * Clone() const override { return new Type(*this); } \
-    static EbmlElement & Create() {return *(new (Type));} \
-    static const EbmlCallbacks ClassInfos; \
-
-
-#define EBML_INFO(ref)             ref::ClassInfos
-#define EBML_ID(ref)               ref::ClassInfos.GlobalId
-#define EBML_CLASS_SEMCONTEXT(ref) Context_##ref
-#define EBML_CLASS_CONTEXT(ref)    ref::ClassInfos.Context
-#define EBML_CLASS_CALLBACK(ref)   ref::ClassInfos
-#define EBML_CONTEXT(e)            (e)->Generic().Context
-#define EBML_NAME(e)               (e)->Generic().DebugName
-
-#define EBML_INFO_ID(cb)      (cb).GlobalId
-#define EBML_INFO_NAME(cb)    (cb).DebugName
-#define EBML_INFO_CREATE(cb)  (cb).Create()
-#define EBML_INFO_CONTEXT(cb) (cb).Context
-
-#define EBML_SEM_UNIQUE(s)  (s).Unique
-#define EBML_SEM_CONTEXT(s) (s).GetCallbacks.Context
-#define EBML_SEM_CREATE(s)  (s).Create()
-
-#define EBML_CTX_SIZE(c)       (c).Size
-#define EBML_CTX_MASTER(c)     (c).MasterElt
-#define EBML_CTX_PARENT(c)     (c).UpTable
-#define EBML_CTX_IDX(c,i)      (c).MyTable[(i)]
-#define EBML_CTX_IDX_INFO(c,i) (c).MyTable[(i)].GetCallbacks
-#define EBML_CTX_IDX_ID(c,i)   (c).MyTable[(i)].GetCallbacks.GlobalId
-#endif
 
 #if !defined(INVALID_FILEPOS_T)
 #define INVALID_FILEPOS_T 0
@@ -298,9 +253,7 @@ class EBML_DLL_API EbmlCallbacks {
         inline const char * GetName() const { return DebugName; }
         inline EbmlElement & NewElement() const { return Create(); }
 
-#if defined(EBML_STRICT_API)
     private:
-#endif
     EbmlElement & (*Create)();
     const EbmlId & GlobalId;
     const char * DebugName;
@@ -321,9 +274,7 @@ class EBML_DLL_API EbmlSemantic {
         inline EbmlElement & Create() const { return EBML_INFO_CREATE(GetCallbacks); }
         inline explicit operator const EbmlCallbacks &() const { return GetCallbacks; }
 
-#if defined(EBML_STRICT_API)
     private:
-#endif
     bool Mandatory; ///< wether the element is mandatory in the context or not
     bool Unique;
     const EbmlCallbacks & GetCallbacks;
@@ -358,9 +309,7 @@ class EBML_DLL_API EbmlSemanticContext {
 
     const _GetSemanticContext GetGlobalContext; ///< global elements supported at this level
 
-#if defined(EBML_STRICT_API)
     private:
-#endif
         const EbmlSemantic *MyTable; ///< First element in the table
     std::size_t Size;          ///< number of elements in the table
     const EbmlSemanticContext *UpTable; ///< Parent element
@@ -399,13 +348,8 @@ class EBML_DLL_API EbmlElement {
     virtual EbmlElement * Clone() const = 0;
 
     virtual explicit operator const EbmlId &() const = 0;
-#if defined(EBML_STRICT_API)
         virtual const char *DebugName() const = 0;
         virtual const EbmlSemanticContext &Context() const = 0;
-#else
-    /// return the generic callback to monitor a derived class
-    virtual const EbmlCallbacks & Generic() const = 0;
-#endif
         virtual EbmlElement & CreateElement() const = 0;
 
     // by default only allow to set element as finite (override when needed)
@@ -501,9 +445,7 @@ class EBML_DLL_API EbmlElement {
         inline void SetSizeIsFinite(bool Set = true) {bSizeIsFinite = Set;}
         inline std::uint64_t GetSizePosition() const {return SizePosition;}
 
-#if defined(EBML_STRICT_API)
   private:
-#endif
     std::uint64_t Size;        ///< the size of the data to write
     std::uint64_t DefaultSize; ///< Minimum data size to fill on rendering (0 = optimal)
     int SizeLength{0}; /// the minimum size on which the size will be written (0 = optimal)

ebml/EbmlEndian.h

@@ -113,11 +113,7 @@ template<class TYPE, endianess ENDIAN> class Endian
       inline std::size_t size() const   { return sizeof(TYPE); }
       inline bool operator!=(const binary *buffer) const {return *(reinterpret_cast<TYPE*>(buffer)) == platform_value;}
 
-#if defined(EBML_STRICT_API)
     private:
-#else
-    protected:
-#endif
       TYPE platform_value;
       TYPE endian_value;
 

ebml/EbmlFloat.h

@@ -92,11 +92,7 @@ class EBML_DLL_API EbmlFloat : public EbmlElement {
       return (DefaultISset() && Value == DefaultValue);
     }
 
-#if defined(EBML_STRICT_API)
     private:
-#else
-    protected:
-#endif
     double Value; /// The actual value of the element
     double DefaultValue;
 };

ebml/EbmlId.h

@@ -41,13 +41,8 @@
 namespace libebml {
 
 
-#if defined(EBML_STRICT_API)
 #define EBML_ID_VALUE(id)  (id).GetValue()
 #define EBML_ID_LENGTH(id) (id).GetLength()
-#else
-#define EBML_ID_VALUE(id)  (id).Value
-#define EBML_ID_LENGTH(id) (id).Length
-#endif
 
 /*!
   \class EbmlId
@@ -86,9 +81,7 @@ class EBML_DLL_API EbmlId {
         inline std::size_t GetLength() const { return Length; }
         inline std::uint32_t GetValue() const { return Value; }
 
-#if defined(EBML_STRICT_API)
     private:
-#endif
     std::uint32_t Value;
     std::size_t Length;
 };

ebml/EbmlMaster.h

@@ -178,11 +178,7 @@ class EBML_DLL_API EbmlMaster : public EbmlElement {
     */
     std::vector<std::string> FindAllMissingElements();
 
-#if defined(EBML_STRICT_API)
     private:
-#else
-    protected:
-#endif
     std::vector<EbmlElement *> ElementList;
 
     const EbmlSemanticContext & Context;

ebml/EbmlSInteger.h

@@ -86,11 +86,7 @@ class EBML_DLL_API EbmlSInteger : public EbmlElement {
       return (DefaultISset() && Value == DefaultValue);
     }
 
-#if defined(EBML_STRICT_API)
     private:
-#else
-    protected:
-#endif
     std::int64_t Value; /// The actual value of the element
     std::int64_t DefaultValue;
 };

ebml/EbmlStream.h

@@ -61,11 +61,7 @@ class EBML_DLL_API EbmlStream {
     inline IOCallback & I_O() {return Stream;}
         operator IOCallback &() {return Stream;}
 
-#if defined(EBML_STRICT_API)
     private:
-#else
-    protected:
-#endif
     IOCallback & Stream;
 };
 

ebml/EbmlString.h

@@ -71,11 +71,7 @@ class EBML_DLL_API EbmlString : public EbmlElement {
       return (DefaultISset() && Value == DefaultValue);
     }
 
-#if defined(EBML_STRICT_API)
     private:
-#else
-    protected:
-#endif
     std::string Value;  /// The actual value of the element
     std::string DefaultValue;
 };

ebml/EbmlUInteger.h

@@ -84,11 +84,7 @@ class EBML_DLL_API EbmlUInteger : public EbmlElement {
       return (DefaultISset() && Value == DefaultValue);
     }
 
-#if defined(EBML_STRICT_API)
     private:
-#else
-    protected:
-#endif
     std::uint64_t Value; /// The actual value of the element
     std::uint64_t DefaultValue;
 };

ebml/EbmlUnicodeString.h

@@ -78,11 +78,7 @@ class EBML_DLL_API UTFstring {
   const std::string & GetUTF8() const {return UTF8string;}
   void SetUTF8(const std::string &);
 
-#if defined(EBML_STRICT_API)
     private:
-#else
-    protected:
-#endif
   std::size_t _Length{0}; ///< length of the UCS string excluding the \0
   wchar_t* _Data{nullptr}; ///< internal UCS representation
   std::string UTF8string;
@@ -123,11 +119,7 @@ class EBML_DLL_API EbmlUnicodeString : public EbmlElement {
       return (DefaultISset() && Value == DefaultValue);
     }
 
-#if defined(EBML_STRICT_API)
     private:
-#else
-    protected:
-#endif
     UTFstring Value; /// The actual value of the element
     UTFstring DefaultValue;
 };