Remove virtual methods QPDFValue::getTypeCode and getTypeName

28 files changed, 46 insertions, 192 deletions

diff --git a/include/qpdf/QPDFObject.hh b/include/qpdf/QPDFObject.hh
index db6efa4c..a1930168 100644
--- a/include/qpdf/QPDFObject.hh
+++ b/include/qpdf/QPDFObject.hh
@@ -87,7 +87,7 @@ class QPDFObject
     object_type_e
     getTypeCode() const
     {
-        return value->getTypeCode();
+        return value->type_code;
     }
 
     // Return a string literal that describes the type, useful for
@@ -95,9 +95,8 @@ class QPDFObject
     char const*
     getTypeName() const
     {
-        return value->getTypeName();
+        return value->type_name;
     }
-
     void
     setDescription(QPDF* qpdf, std::string const& description)
     {
diff --git a/include/qpdf/QPDFValue.hh b/include/qpdf/QPDFValue.hh
index b957b813..33558f1b 100644
--- a/include/qpdf/QPDFValue.hh
+++ b/include/qpdf/QPDFValue.hh
@@ -43,8 +43,6 @@ class QPDFValue
     virtual std::shared_ptr<QPDFObject> shallowCopy() = 0;
     virtual std::string unparse() = 0;
     virtual JSON getJSON(int json_version) = 0;
-    virtual qpdf_object_type_e getTypeCode() const = 0;
-    virtual char const* getTypeName() const = 0;
     virtual void
     setDescription(QPDF* qpdf, std::string const& description)
     {
@@ -75,7 +73,17 @@ class QPDFValue
     }
 
   protected:
-    QPDFValue() = default;
+    QPDFValue() :
+        type_code(::ot_uninitialized),
+        type_name("uninitilized")
+    {
+    }
+    QPDFValue(qpdf_object_type_e type_code, char const* type_name) :
+        type_code(type_code),
+        type_name(type_name)
+    {
+    }
+
     virtual void
     releaseResolved()
     {
@@ -88,6 +96,8 @@ class QPDFValue
     QPDF* owning_qpdf{nullptr};
     std::string object_description;
     qpdf_offset_t parsed_offset{-1};
+    const qpdf_object_type_e type_code;
+    char const* type_name;
 };
 
 #endif // QPDFVALUE_HH
diff --git a/libqpdf/QPDF_Array.cc b/libqpdf/QPDF_Array.cc
index ff35e9a3..63fe98d4 100644
--- a/libqpdf/QPDF_Array.cc
+++ b/libqpdf/QPDF_Array.cc
@@ -4,12 +4,14 @@
 #include <qpdf/QUtil.hh>
 #include <stdexcept>
 
-QPDF_Array::QPDF_Array(std::vector<QPDFObjectHandle> const& v)
+QPDF_Array::QPDF_Array(std::vector<QPDFObjectHandle> const& v) :
+    QPDFValue(::ot_array, "array")
 {
     setFromVector(v);
 }
 
 QPDF_Array::QPDF_Array(SparseOHArray const& items) :
+    QPDFValue(::ot_array, "array"),
     elements(items)
 {
 }
@@ -62,18 +64,6 @@ QPDF_Array::getJSON(int json_version)
     return j;
 }
 
-qpdf_object_type_e
-QPDF_Array::getTypeCode() const
-{
-    return ::ot_array;
-}
-
-char const*
-QPDF_Array::getTypeName() const
-{
-    return "array";
-}
-
 int
 QPDF_Array::getNItems() const
 {
diff --git a/libqpdf/QPDF_Bool.cc b/libqpdf/QPDF_Bool.cc
index e58eb62b..efbfd6c9 100644
--- a/libqpdf/QPDF_Bool.cc
+++ b/libqpdf/QPDF_Bool.cc
@@ -1,6 +1,7 @@
 #include <qpdf/QPDF_Bool.hh>
 
 QPDF_Bool::QPDF_Bool(bool val) :
+    QPDFValue(::ot_boolean, "boolean"),
     val(val)
 {
 }
@@ -29,18 +30,6 @@ QPDF_Bool::getJSON(int json_version)
     return JSON::makeBool(this->val);
 }
 
-qpdf_object_type_e
-QPDF_Bool::getTypeCode() const
-{
-    return ::ot_boolean;
-}
-
-char const*
-QPDF_Bool::getTypeName() const
-{
-    return "boolean";
-}
-
 bool
 QPDF_Bool::getVal() const
 {
diff --git a/libqpdf/QPDF_Dictionary.cc b/libqpdf/QPDF_Dictionary.cc
index 7f374f81..845bcad8 100644
--- a/libqpdf/QPDF_Dictionary.cc
+++ b/libqpdf/QPDF_Dictionary.cc
@@ -4,6 +4,7 @@
 
 QPDF_Dictionary::QPDF_Dictionary(
     std::map<std::string, QPDFObjectHandle> const& items) :
+    QPDFValue(::ot_dictionary, "dictionary"),
     items(items)
 {
 }
@@ -57,18 +58,6 @@ QPDF_Dictionary::getJSON(int json_version)
     return j;
 }
 
-qpdf_object_type_e
-QPDF_Dictionary::getTypeCode() const
-{
-    return ::ot_dictionary;
-}
-
-char const*
-QPDF_Dictionary::getTypeName() const
-{
-    return "dictionary";
-}
-
 bool
 QPDF_Dictionary::hasKey(std::string const& key)
 {
diff --git a/libqpdf/QPDF_InlineImage.cc b/libqpdf/QPDF_InlineImage.cc
index 494363f6..76318196 100644
--- a/libqpdf/QPDF_InlineImage.cc
+++ b/libqpdf/QPDF_InlineImage.cc
@@ -1,6 +1,7 @@
 #include <qpdf/QPDF_InlineImage.hh>
 
 QPDF_InlineImage::QPDF_InlineImage(std::string const& val) :
+    QPDFValue(::ot_inlineimage, "inline-image"),
     val(val)
 {
 }
@@ -29,18 +30,6 @@ QPDF_InlineImage::getJSON(int json_version)
     return JSON::makeNull();
 }
 
-qpdf_object_type_e
-QPDF_InlineImage::getTypeCode() const
-{
-    return ::ot_inlineimage;
-}
-
-char const*
-QPDF_InlineImage::getTypeName() const
-{
-    return "inline-image";
-}
-
 std::string
 QPDF_InlineImage::getVal() const
 {
diff --git a/libqpdf/QPDF_Integer.cc b/libqpdf/QPDF_Integer.cc
index 68f97420..24812573 100644
--- a/libqpdf/QPDF_Integer.cc
+++ b/libqpdf/QPDF_Integer.cc
@@ -3,6 +3,7 @@
 #include <qpdf/QUtil.hh>
 
 QPDF_Integer::QPDF_Integer(long long val) :
+    QPDFValue(::ot_integer, "integer"),
     val(val)
 {
 }
@@ -31,18 +32,6 @@ QPDF_Integer::getJSON(int json_version)
     return JSON::makeInt(this->val);
 }
 
-qpdf_object_type_e
-QPDF_Integer::getTypeCode() const
-{
-    return ::ot_integer;
-}
-
-char const*
-QPDF_Integer::getTypeName() const
-{
-    return "integer";
-}
-
 long long
 QPDF_Integer::getVal() const
 {
diff --git a/libqpdf/QPDF_Name.cc b/libqpdf/QPDF_Name.cc
index 6831bcba..c86d34b4 100644
--- a/libqpdf/QPDF_Name.cc
+++ b/libqpdf/QPDF_Name.cc
@@ -5,6 +5,7 @@
 #include <string.h>
 
 QPDF_Name::QPDF_Name(std::string const& name) :
+    QPDFValue(::ot_name, "name"),
     name(name)
 {
 }
@@ -61,18 +62,6 @@ QPDF_Name::getJSON(int json_version)
     }
 }
 
-qpdf_object_type_e
-QPDF_Name::getTypeCode() const
-{
-    return ::ot_name;
-}
-
-char const*
-QPDF_Name::getTypeName() const
-{
-    return "name";
-}
-
 std::string
 QPDF_Name::getName() const
 {
diff --git a/libqpdf/QPDF_Null.cc b/libqpdf/QPDF_Null.cc
index 67b564c8..f60dda1f 100644
--- a/libqpdf/QPDF_Null.cc
+++ b/libqpdf/QPDF_Null.cc
@@ -1,5 +1,10 @@
 #include <qpdf/QPDF_Null.hh>
 
+QPDF_Null::QPDF_Null() :
+    QPDFValue(::ot_null, "null")
+{
+}
+
 std::shared_ptr<QPDFObject>
 QPDF_Null::create()
 {
@@ -23,15 +28,3 @@ QPDF_Null::getJSON(int json_version)
 {
     return JSON::makeNull();
 }
-
-qpdf_object_type_e
-QPDF_Null::getTypeCode() const
-{
-    return ::ot_null;
-}
-
-char const*
-QPDF_Null::getTypeName() const
-{
-    return "null";
-}
diff --git a/libqpdf/QPDF_Operator.cc b/libqpdf/QPDF_Operator.cc
index 84bde381..547ff40a 100644
--- a/libqpdf/QPDF_Operator.cc
+++ b/libqpdf/QPDF_Operator.cc
@@ -1,6 +1,7 @@
 #include <qpdf/QPDF_Operator.hh>
 
 QPDF_Operator::QPDF_Operator(std::string const& val) :
+    QPDFValue(::ot_operator, "operator"),
     val(val)
 {
 }
@@ -29,18 +30,6 @@ QPDF_Operator::getJSON(int json_version)
     return JSON::makeNull();
 }
 
-qpdf_object_type_e
-QPDF_Operator::getTypeCode() const
-{
-    return ::ot_operator;
-}
-
-char const*
-QPDF_Operator::getTypeName() const
-{
-    return "operator";
-}
-
 std::string
 QPDF_Operator::getVal() const
 {
diff --git a/libqpdf/QPDF_Real.cc b/libqpdf/QPDF_Real.cc
index 1604d4ba..85c9ceeb 100644
--- a/libqpdf/QPDF_Real.cc
+++ b/libqpdf/QPDF_Real.cc
@@ -3,12 +3,14 @@
 #include <qpdf/QUtil.hh>
 
 QPDF_Real::QPDF_Real(std::string const& val) :
+    QPDFValue(::ot_real, "real"),
     val(val)
 {
 }
 
 QPDF_Real::QPDF_Real(
     double value, int decimal_places, bool trim_trailing_zeroes) :
+    QPDFValue(::ot_real, "real"),
     val(QUtil::double_to_string(value, decimal_places, trim_trailing_zeroes))
 {
 }
@@ -60,18 +62,6 @@ QPDF_Real::getJSON(int json_version)
     return JSON::makeNumber(result);
 }
 
-qpdf_object_type_e
-QPDF_Real::getTypeCode() const
-{
-    return ::ot_real;
-}
-
-char const*
-QPDF_Real::getTypeName() const
-{
-    return "real";
-}
-
 std::string
 QPDF_Real::getVal()
 {
diff --git a/libqpdf/QPDF_Reserved.cc b/libqpdf/QPDF_Reserved.cc
index 14f82a62..f5af4688 100644
--- a/libqpdf/QPDF_Reserved.cc
+++ b/libqpdf/QPDF_Reserved.cc
@@ -2,6 +2,11 @@
 
 #include <stdexcept>
 
+QPDF_Reserved::QPDF_Reserved() :
+    QPDFValue(::ot_reserved, "reserved")
+{
+}
+
 std::shared_ptr<QPDFObject>
 QPDF_Reserved::create()
 {
@@ -29,15 +34,3 @@ QPDF_Reserved::getJSON(int json_version)
         "QPDFObjectHandle: attempting to unparse a reserved object");
     return JSON::makeNull();
 }
-
-qpdf_object_type_e
-QPDF_Reserved::getTypeCode() const
-{
-    return ::ot_reserved;
-}
-
-char const*
-QPDF_Reserved::getTypeName() const
-{
-    return "reserved";
-}
diff --git a/libqpdf/QPDF_Stream.cc b/libqpdf/QPDF_Stream.cc
index 7a74512f..9932c15d 100644
--- a/libqpdf/QPDF_Stream.cc
+++ b/libqpdf/QPDF_Stream.cc
@@ -114,6 +114,7 @@ QPDF_Stream::QPDF_Stream(
     QPDFObjectHandle stream_dict,
     qpdf_offset_t offset,
     size_t length) :
+    QPDFValue(::ot_stream, "stream"),
     qpdf(qpdf),
     og(og),
     filter_on_write(true),
@@ -291,18 +292,6 @@ QPDF_Stream::getStreamJSON(
     return result;
 }
 
-qpdf_object_type_e
-QPDF_Stream::getTypeCode() const
-{
-    return ::ot_stream;
-}
-
-char const*
-QPDF_Stream::getTypeName() const
-{
-    return "stream";
-}
-
 void
 QPDF_Stream::setDescription(QPDF* qpdf, std::string const& description)
 {
diff --git a/libqpdf/QPDF_String.cc b/libqpdf/QPDF_String.cc
index c8e21463..c6cb6c41 100644
--- a/libqpdf/QPDF_String.cc
+++ b/libqpdf/QPDF_String.cc
@@ -21,6 +21,7 @@ is_iso_latin1_printable(char ch)
 }
 
 QPDF_String::QPDF_String(std::string const& val) :
+    QPDFValue(::ot_string, "string"),
     val(val)
 {
 }
@@ -84,18 +85,6 @@ QPDF_String::getJSON(int json_version)
     return JSON::makeString(result);
 }
 
-qpdf_object_type_e
-QPDF_String::getTypeCode() const
-{
-    return ::ot_string;
-}
-
-char const*
-QPDF_String::getTypeName() const
-{
-    return "string";
-}
-
 bool
 QPDF_String::useHexString() const
 {
diff --git a/libqpdf/QPDF_Unresolved.cc b/libqpdf/QPDF_Unresolved.cc
index b51075f7..40d4874e 100644
--- a/libqpdf/QPDF_Unresolved.cc
+++ b/libqpdf/QPDF_Unresolved.cc
@@ -2,6 +2,11 @@
 
 #include <stdexcept>
 
+QPDF_Unresolved::QPDF_Unresolved() :
+    QPDFValue(::ot_unresolved, "unresolved")
+{
+}
+
 std::shared_ptr<QPDFObject>
 QPDF_Unresolved::create()
 {
@@ -27,15 +32,3 @@ QPDF_Unresolved::getJSON(int json_version)
 {
     return JSON::makeNull();
 }
-
-qpdf_object_type_e
-QPDF_Unresolved::getTypeCode() const
-{
-    return ::ot_unresolved;
-}
-
-char const*
-QPDF_Unresolved::getTypeName() const
-{
-    return "unresolved";
-}
diff --git a/libqpdf/qpdf/QPDF_Array.hh b/libqpdf/qpdf/QPDF_Array.hh
index b0207115..426efe36 100644
--- a/libqpdf/qpdf/QPDF_Array.hh
+++ b/libqpdf/qpdf/QPDF_Array.hh
@@ -17,8 +17,6 @@ class QPDF_Array: public QPDFValue
     virtual std::shared_ptr<QPDFObject> shallowCopy();
     virtual std::string unparse();
     virtual JSON getJSON(int json_version);
-    virtual qpdf_object_type_e getTypeCode() const;
-    virtual char const* getTypeName() const;
 
     int getNItems() const;
     QPDFObjectHandle getItem(int n) const;
diff --git a/libqpdf/qpdf/QPDF_Bool.hh b/libqpdf/qpdf/QPDF_Bool.hh
index 2de585cb..3e45cd8e 100644
--- a/libqpdf/qpdf/QPDF_Bool.hh
+++ b/libqpdf/qpdf/QPDF_Bool.hh
@@ -11,8 +11,6 @@ class QPDF_Bool: public QPDFValue
     virtual std::shared_ptr<QPDFObject> shallowCopy();
     virtual std::string unparse();
     virtual JSON getJSON(int json_version);
-    virtual qpdf_object_type_e getTypeCode() const;
-    virtual char const* getTypeName() const;
     bool getVal() const;
 
   private:
diff --git a/libqpdf/qpdf/QPDF_Dictionary.hh b/libqpdf/qpdf/QPDF_Dictionary.hh
index 5761b030..19ab8d9b 100644
--- a/libqpdf/qpdf/QPDF_Dictionary.hh
+++ b/libqpdf/qpdf/QPDF_Dictionary.hh
@@ -17,8 +17,6 @@ class QPDF_Dictionary: public QPDFValue
     virtual std::shared_ptr<QPDFObject> shallowCopy();
     virtual std::string unparse();
     virtual JSON getJSON(int json_version);
-    virtual qpdf_object_type_e getTypeCode() const;
-    virtual char const* getTypeName() const;
 
     // hasKey() and getKeys() treat keys with null values as if they
     // aren't there.  getKey() returns null for the value of a
diff --git a/libqpdf/qpdf/QPDF_InlineImage.hh b/libqpdf/qpdf/QPDF_InlineImage.hh
index 31f7529a..b7bea9c7 100644
--- a/libqpdf/qpdf/QPDF_InlineImage.hh
+++ b/libqpdf/qpdf/QPDF_InlineImage.hh
@@ -11,8 +11,6 @@ class QPDF_InlineImage: public QPDFValue
     virtual std::shared_ptr<QPDFObject> shallowCopy();
     virtual std::string unparse();
     virtual JSON getJSON(int json_version);
-    virtual qpdf_object_type_e getTypeCode() const;
-    virtual char const* getTypeName() const;
     std::string getVal() const;
 
   private:
diff --git a/libqpdf/qpdf/QPDF_Integer.hh b/libqpdf/qpdf/QPDF_Integer.hh
index cf201021..7e09673c 100644
--- a/libqpdf/qpdf/QPDF_Integer.hh
+++ b/libqpdf/qpdf/QPDF_Integer.hh
@@ -11,8 +11,6 @@ class QPDF_Integer: public QPDFValue
     virtual std::shared_ptr<QPDFObject> shallowCopy();
     virtual std::string unparse();
     virtual JSON getJSON(int json_version);
-    virtual qpdf_object_type_e getTypeCode() const;
-    virtual char const* getTypeName() const;
     long long getVal() const;
 
   private:
diff --git a/libqpdf/qpdf/QPDF_Name.hh b/libqpdf/qpdf/QPDF_Name.hh
index 438b118e..74fc7e44 100644
--- a/libqpdf/qpdf/QPDF_Name.hh
+++ b/libqpdf/qpdf/QPDF_Name.hh
@@ -11,8 +11,6 @@ class QPDF_Name: public QPDFValue
     virtual std::shared_ptr<QPDFObject> shallowCopy();
     virtual std::string unparse();
     virtual JSON getJSON(int json_version);
-    virtual qpdf_object_type_e getTypeCode() const;
-    virtual char const* getTypeName() const;
     std::string getName() const;
 
     // Put # into strings with characters unsuitable for name token
diff --git a/libqpdf/qpdf/QPDF_Null.hh b/libqpdf/qpdf/QPDF_Null.hh
index 7f722a3a..68973de9 100644
--- a/libqpdf/qpdf/QPDF_Null.hh
+++ b/libqpdf/qpdf/QPDF_Null.hh
@@ -11,11 +11,9 @@ class QPDF_Null: public QPDFValue
     virtual std::shared_ptr<QPDFObject> shallowCopy();
     virtual std::string unparse();
     virtual JSON getJSON(int json_version);
-    virtual qpdf_object_type_e getTypeCode() const;
-    virtual char const* getTypeName() const;
 
   private:
-    QPDF_Null() = default;
+    QPDF_Null();
 };
 
 #endif // QPDF_NULL_HH
diff --git a/libqpdf/qpdf/QPDF_Operator.hh b/libqpdf/qpdf/QPDF_Operator.hh
index d17f1629..767c0ba0 100644
--- a/libqpdf/qpdf/QPDF_Operator.hh
+++ b/libqpdf/qpdf/QPDF_Operator.hh
@@ -11,8 +11,6 @@ class QPDF_Operator: public QPDFValue
     virtual std::shared_ptr<QPDFObject> shallowCopy();
     virtual std::string unparse();
     virtual JSON getJSON(int json_version);
-    virtual qpdf_object_type_e getTypeCode() const;
-    virtual char const* getTypeName() const;
     std::string getVal() const;
 
   private:
diff --git a/libqpdf/qpdf/QPDF_Real.hh b/libqpdf/qpdf/QPDF_Real.hh
index 97375c73..dc0f3ff8 100644
--- a/libqpdf/qpdf/QPDF_Real.hh
+++ b/libqpdf/qpdf/QPDF_Real.hh
@@ -13,8 +13,6 @@ class QPDF_Real: public QPDFValue
     virtual std::shared_ptr<QPDFObject> shallowCopy();
     virtual std::string unparse();
     virtual JSON getJSON(int json_version);
-    virtual qpdf_object_type_e getTypeCode() const;
-    virtual char const* getTypeName() const;
     std::string getVal();
 
   private:
diff --git a/libqpdf/qpdf/QPDF_Reserved.hh b/libqpdf/qpdf/QPDF_Reserved.hh
index e9f8313a..f90242a9 100644
--- a/libqpdf/qpdf/QPDF_Reserved.hh
+++ b/libqpdf/qpdf/QPDF_Reserved.hh
@@ -11,11 +11,9 @@ class QPDF_Reserved: public QPDFValue
     virtual std::shared_ptr<QPDFObject> shallowCopy();
     virtual std::string unparse();
     virtual JSON getJSON(int json_version);
-    virtual qpdf_object_type_e getTypeCode() const;
-    virtual char const* getTypeName() const;
 
   private:
-    QPDF_Reserved() = default;
+    QPDF_Reserved();
 };
 
 #endif // QPDF_RESERVED_HH
diff --git a/libqpdf/qpdf/QPDF_Stream.hh b/libqpdf/qpdf/QPDF_Stream.hh
index 639d5298..3a16160e 100644
--- a/libqpdf/qpdf/QPDF_Stream.hh
+++ b/libqpdf/qpdf/QPDF_Stream.hh
@@ -26,8 +26,6 @@ class QPDF_Stream: public QPDFValue
     virtual std::shared_ptr<QPDFObject> shallowCopy();
     virtual std::string unparse();
     virtual JSON getJSON(int json_version);
-    virtual qpdf_object_type_e getTypeCode() const;
-    virtual char const* getTypeName() const;
     virtual void setDescription(QPDF*, std::string const&);
     QPDFObjectHandle getDict() const;
     bool isDataModified() const;
diff --git a/libqpdf/qpdf/QPDF_String.hh b/libqpdf/qpdf/QPDF_String.hh
index 9b6fc95c..a92427e3 100644
--- a/libqpdf/qpdf/QPDF_String.hh
+++ b/libqpdf/qpdf/QPDF_String.hh
@@ -16,8 +16,6 @@ class QPDF_String: public QPDFValue
     create_utf16(std::string const& utf8_val);
     virtual std::shared_ptr<QPDFObject> shallowCopy();
     virtual std::string unparse();
-    virtual qpdf_object_type_e getTypeCode() const;
-    virtual char const* getTypeName() const;
     std::string unparse(bool force_binary);
     virtual JSON getJSON(int json_version);
     std::string getVal() const;
diff --git a/libqpdf/qpdf/QPDF_Unresolved.hh b/libqpdf/qpdf/QPDF_Unresolved.hh
index c17748b5..c1231590 100644
--- a/libqpdf/qpdf/QPDF_Unresolved.hh
+++ b/libqpdf/qpdf/QPDF_Unresolved.hh
@@ -11,11 +11,9 @@ class QPDF_Unresolved: public QPDFValue
     virtual std::shared_ptr<QPDFObject> shallowCopy();
     virtual std::string unparse();
     virtual JSON getJSON(int json_version);
-    virtual qpdf_object_type_e getTypeCode() const;
-    virtual char const* getTypeName() const;
 
   private:
-    QPDF_Unresolved() = default;
+    QPDF_Unresolved();
 };
 
 #endif // QPDF_UNRESOLVED_HH