diff options
| author | Keith Isdale <keith.isdale@nokia.com> | 2009-07-01 02:35:46 (GMT) |
|---|---|---|
| committer | Keith Isdale <keith.isdale@nokia.com> | 2009-08-24 05:28:07 (GMT) |
| commit | fefd6e45962ac915fe9cd43b245642985c36b36d (patch) | |
| tree | 4d0694fcb0720d493b7bb4995533865535c32675 /src/xmlpatterns | |
| parent | 6c282e096c160ef0426466042bf4bb4e0e28f0b2 (diff) | |
| download | Qt-fefd6e45962ac915fe9cd43b245642985c36b36d.zip Qt-fefd6e45962ac915fe9cd43b245642985c36b36d.tar.gz Qt-fefd6e45962ac915fe9cd43b245642985c36b36d.tar.bz2 | |
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
Diffstat (limited to 'src/xmlpatterns')
| -rw-r--r-- | src/xmlpatterns/acceltree/qacceltree_p.h | 1 | ||||
| -rw-r--r-- | src/xmlpatterns/api/qabstractxmlforwarditerator_p.h | 2 |
2 files changed, 2 insertions, 1 deletions
diff --git a/src/xmlpatterns/acceltree/qacceltree_p.h b/src/xmlpatterns/acceltree/qacceltree_p.h index 38f5a8e..a033c15 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 a0f7131..fde5634 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; |