diff options
author | Shawn Rutledge <shawn.rutledge@qt.io> | 2019-12-23 08:39:26 +0100 |
---|---|---|
committer | Shawn Rutledge <shawn.rutledge@qt.io> | 2019-12-31 18:15:05 +0100 |
commit | 289c0fdc2cafd79e3178a9cd3cfc6c5bd53c8336 (patch) | |
tree | 6dd5ba0381560b640d0ef89c31ae1e7bc434c546 /src/quick/items/qquickimage_p_p.h | |
parent | dbde4265657db16621551d12a25ff8ffaa18eb73 (diff) |
Optimize QQuickImagePrivate padding
Since the parent QQuickImageBasePrivate class ends with a bool bitfield,
starting QQuickImagePrivate with its own bitfield seems to cause them to
dovetail: at least the warnings about padding are eliminated. It's
anyway nice to group variables by size (although in reverse order),
group methods separately, and do header initialization as much as
possible.
Change-Id: I934e4ad682e338ebb62721ed40c60ff9d4465600
Reviewed-by: Shawn Rutledge <shawn.rutledge@qt.io>
Diffstat (limited to 'src/quick/items/qquickimage_p_p.h')
-rw-r--r-- | src/quick/items/qquickimage_p_p.h | 13 |
1 files changed, 6 insertions, 7 deletions
diff --git a/src/quick/items/qquickimage_p_p.h b/src/quick/items/qquickimage_p_p.h index e2c467c311..1c4ba88655 100644 --- a/src/quick/items/qquickimage_p_p.h +++ b/src/quick/items/qquickimage_p_p.h @@ -80,19 +80,18 @@ class Q_QUICK_PRIVATE_EXPORT QQuickImagePrivate : public QQuickImageBasePrivate public: QQuickImagePrivate(); - - QQuickImage::FillMode fillMode; - qreal paintedWidth; - qreal paintedHeight; void setImage(const QImage &img); void setPixmap(const QQuickPixmap &pixmap); bool pixmapChanged : 1; bool mipmap : 1; - QQuickImage::HAlignment hAlign; - QQuickImage::VAlignment vAlign; + QQuickImage::HAlignment hAlign = QQuickImage::AlignHCenter; + QQuickImage::VAlignment vAlign = QQuickImage::AlignVCenter; + QQuickImage::FillMode fillMode = QQuickImage::Stretch; - QQuickImageTextureProvider *provider; + qreal paintedWidth = 0; + qreal paintedHeight = 0; + QQuickImageTextureProvider *provider = nullptr; }; QT_END_NAMESPACE |