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/svg | |
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/svg')
-rw-r--r-- | src/svg/qsvgstyle.cpp | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/src/svg/qsvgstyle.cpp b/src/svg/qsvgstyle.cpp index 720dbb6..4cbd49a 100644 --- a/src/svg/qsvgstyle.cpp +++ b/src/svg/qsvgstyle.cpp @@ -81,12 +81,14 @@ void QSvgQualityStyle::revert(QPainter *, QSvgExtraStates &) } QSvgFillStyle::QSvgFillStyle(const QBrush &brush) - : m_fill(brush), m_style(0), m_fillRuleSet(false), m_fillOpacitySet(false) + : m_fill(brush), m_style(0), m_fillRuleSet(false), m_fillRule(Qt::OddEvenFill), + m_fillOpacitySet(false), m_fillOpacity(0),m_oldOpacity(0) { } QSvgFillStyle::QSvgFillStyle(QSvgStyleProperty *style) - : m_style(style), m_fillRuleSet(false), m_fillOpacitySet(false) + : m_style(style), m_fillRuleSet(false), m_fillRule(Qt::OddEvenFill), + m_fillOpacitySet(false), m_fillOpacity(0),m_oldOpacity(0) { } |