Ord items list now contain pointers to the Items

so that it can contain subclasses of Item, since the ones contained are
just the pointers. The space reserved for an Item wouldn't be enough for
a subclass' additional members. The space reserved for *a pointer to* an
Item, on the other hand, should be enough for a later casted pointer to
something derived from an Item.

1 files changed, 7 insertions, 4 deletions

diff --git a/interchange/ord.hxx b/interchange/ord.hxx
index 302e6db..390d415 100644
--- a/interchange/ord.hxx
+++ b/interchange/ord.hxx
@@ -48,7 +48,7 @@ namespace QInterchange {
 			Ord() {}
 			explicit Ord(struct interchange_ord_order *order,
 					QObject* parent = nullptr);
-			~Ord() {}
+			virtual ~Ord() { for (auto item : items) delete item; }
 			int rowCount(QModelIndex const& parent = QModelIndex()) const Q_DECL_OVERRIDE;
 			QVariant data(const QModelIndex& index, int role = Qt::DisplayRole) const
 				Q_DECL_OVERRIDE;
@@ -76,13 +76,16 @@ namespace QInterchange {
 		protected:
 			QHash<int, QByteArray> roleNames() const Q_DECL_OVERRIDE;
 			void init(struct interchange_ord_order *order);
-			void addItem(Item const &item);
-			const Item &itemAt(int row) const { return items[row]; }
+			void addItem(Item *item);
+			const Item *itemAt(int row) const
+			{
+				return items.at(row);
+			}
 			void emitUpdate(const QString &response);
 			void emitTransaction(QString const& response);
 
 		private:
-			QList<Item> items;
+			QList<Item *> items;
 			QString profile;
 			double m_subtotal;
 			double m_shipping;