Fixed|libdeng2: de::Reader was missing a destructor

Reader's internal Instance was never freed.
skyjake · Dec 4, 2012 · 99e83ff · 99e83ff
1 parent 16cd87a
commit 99e83ff
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 0 deletions.
diff --git a/doomsday/libdeng2/include/de/data/reader.h b/doomsday/libdeng2/include/de/data/reader.h
@@ -83,6 +83,8 @@ class DENG2_PUBLIC Reader
      */
     Reader(IIStream const &stream, ByteOrder const &byteOrder = littleEndianByteOrder);
 
+    virtual ~Reader();
+
     //@{ Read a number from the source buffer, in network byte order.
     Reader &operator >> (char &byte);
     Reader &operator >> (dchar &byte);

diff --git a/doomsday/libdeng2/src/data/reader.cpp b/doomsday/libdeng2/src/data/reader.cpp
@@ -193,6 +193,11 @@ Reader::Reader(IIStream const &stream, ByteOrder const &byteOrder)
     : d(new Instance(byteOrder, &stream))
 {}
 
+Reader::~Reader()
+{
+    delete d;
+}
+
 Reader &Reader::operator >> (char &byte)
 {
     return *this >> reinterpret_cast<duchar &>(byte);