ElementMessage also needs a valid structure

The solution is the same as for ApplicationMessage, and
since it wasn't caught by the unit tests add a unit test.

2 files changed, 14 insertions, 1 deletions

diff --git a/src/QGst/message.cpp b/src/QGst/message.cpp
index 3e7bb56..e8b5a90 100644
--- a/src/QGst/message.cpp
+++ b/src/QGst/message.cpp
@@ -356,7 +356,7 @@ ApplicationMessagePtr ApplicationMessage::create(const ObjectPtr & source, const
 
 ElementMessagePtr ElementMessage::create(const ObjectPtr & source, const Structure & structure)
 {
-    GstStructure *s = structure.isValid() ? gst_structure_copy(structure) : NULL;
+    GstStructure *s = structure.isValid() ? gst_structure_copy(structure) : gst_structure_new_empty("null");
     return ElementMessagePtr::wrap(gst_message_new_element(source, s), false);
 }
 
diff --git a/tests/auto/refpointertest.cpp b/tests/auto/refpointertest.cpp
index 7a509b7..6fe065a 100644
--- a/tests/auto/refpointertest.cpp
+++ b/tests/auto/refpointertest.cpp
@@ -182,6 +182,19 @@ void RefPointerTest::cppWrappersTest()
                 != static_cast<QGlib::RefCountedObject*>(msg2.operator->()));
         QVERIFY(msg3 == msg2);
     }
+
+    {
+        QGst::MessagePtr msg = QGst::ElementMessage::create(e);
+        QGst::MessagePtr msg2 = msg;
+        QCOMPARE(static_cast<QGlib::RefCountedObject*>(msg.operator->()),
+                 static_cast<QGlib::RefCountedObject*>(msg2.operator->()));
+        QVERIFY(msg2 == msg);
+
+        QGst::MessagePtr msg3 = QGst::MessagePtr::wrap(msg2);
+        QVERIFY(static_cast<QGlib::RefCountedObject*>(msg3.operator->())
+                != static_cast<QGlib::RefCountedObject*>(msg2.operator->()));
+        QVERIFY(msg3 == msg2);
+    }   
 }
 
 void RefPointerTest::messageDynamicCastTest()