Make assorted constructors follow good coding practice

Ensure that class members are initialized
Moved some value assignment from constructor body into the the constructor's intializer list

Reviewed-by: Jason McDonald

2 files changed, 2 insertions, 1 deletions

diff --git a/src/xmlpatterns/acceltree/qacceltree_p.h b/src/xmlpatterns/acceltree/qacceltree_p.h
index 38f5a8e64c..a033c15415 100644
--- a/src/xmlpatterns/acceltree/qacceltree_p.h
+++ b/src/xmlpatterns/acceltree/qacceltree_p.h
@@ -129,6 +129,7 @@ namespace QPatternist
         {
         public:
             inline BasicNodeData()
+                : m_parent(0), m_size(0), m_depth(0), m_kind(0)
             {
             }
 
diff --git a/src/xmlpatterns/api/qabstractxmlforwarditerator_p.h b/src/xmlpatterns/api/qabstractxmlforwarditerator_p.h
index a0f7131a80..fde5634ad3 100644
--- a/src/xmlpatterns/api/qabstractxmlforwarditerator_p.h
+++ b/src/xmlpatterns/api/qabstractxmlforwarditerator_p.h
@@ -86,7 +86,7 @@ public:
     typedef QList<QExplicitlySharedDataPointer<QAbstractXmlForwardIterator<T> > > List;
     typedef QVector<QExplicitlySharedDataPointer<QAbstractXmlForwardIterator<T> > > Vector;
 
-    inline QAbstractXmlForwardIterator() {}
+    inline QAbstractXmlForwardIterator() : d_ptr(0) {}
     virtual ~QAbstractXmlForwardIterator() {}
 
     virtual T next() = 0;