From f288c5fec572a4dac85e10eaba397da146f219c1 Mon Sep 17 00:00:00 2001
From: Steve Lhomme <slhomme@matroska.org>
Date: Sat, 24 Feb 2024 11:06:38 +0100
Subject: [PATCH] make GetSemantic() constexpr

---
 ebml/EbmlElement.h |  9 ++++++++-
 src/EbmlMaster.cpp | 12 ------------
 2 files changed, 8 insertions(+), 13 deletions(-)

diff --git a/ebml/EbmlElement.h b/ebml/EbmlElement.h
index 1952f9d7..ff5bc7cb 100644
--- a/ebml/EbmlElement.h
+++ b/ebml/EbmlElement.h
@@ -17,6 +17,7 @@
 #include <string>
 #include <limits>
 #include <cstddef>
+#include <stdexcept>
 
 namespace libebml {
 
@@ -526,7 +527,13 @@ class EBML_DLL_API EbmlSemanticContextMaster : public EbmlSemanticContext {
       return false;
     }
 
-        const EbmlSemantic & GetSemantic(std::size_t i) const;
+    inline constexpr const EbmlSemantic & GetSemantic(std::size_t i) const
+    {
+      if (i<GetSize())
+        return MyTable[i];
+
+      throw std::logic_error("EbmlSemanticContext::GetSemantic: programming error: index outside of table size");
+    }
 
     private:
         const EbmlSemantic *MyTable; ///< First element in the table
diff --git a/src/EbmlMaster.cpp b/src/EbmlMaster.cpp
index 87ea59f3..750b273a 100644
--- a/src/EbmlMaster.cpp
+++ b/src/EbmlMaster.cpp
@@ -43,18 +43,6 @@ EbmlMaster::~EbmlMaster()
   }
 }
 
-const EbmlSemantic & EbmlSemanticContextMaster::GetSemantic(std::size_t i) const
-{
-  assert(i<GetSize());
-  if (i<GetSize())
-    return MyTable[i];
-
-  std::stringstream ss;
-  ss << "EbmlSemanticContext::GetSemantic: programming error: index i outside of table size (" << i << " >= " << GetSize() << ")";
-  throw std::logic_error(ss.str());
-}
-
-
 /*!
   \todo handle exception on errors
   \todo write all the Mandatory elements in the Context, otherwise assert