diff options
author | Yuhang Zhao <2546789017@qq.com> | 2022-12-20 10:04:51 +0800 |
---|---|---|
committer | Yuhang Zhao <2546789017@qq.com> | 2022-12-20 20:53:02 +0000 |
commit | 528f28f380e3f0c55b9a305f6b208dc1a2db73d6 (patch) | |
tree | a423eaacf6875f933f0f86f94369689276be4bdd /src/gui/kernel/qstylehints.cpp | |
parent | f21c4500a6d4787d679d13e859b34d6c6a7a8c0e (diff) |
QStyleHints: code tidies
1. Use Q_D and Q_Q instead of d_func() and q_func().
2. Use early return.
3. Remove unnecessary semicon.
4. Add one new line to separate two unrelated functions.
Change-Id: Ie6171c857f00ca171e97723a0d517d2295bbbbd1
Reviewed-by: Axel Spoerl <axel.spoerl@qt.io>
Reviewed-by: Paul Wicking <paul.wicking@qt.io>
Diffstat (limited to 'src/gui/kernel/qstylehints.cpp')
-rw-r--r-- | src/gui/kernel/qstylehints.cpp | 12 |
1 files changed, 7 insertions, 5 deletions
diff --git a/src/gui/kernel/qstylehints.cpp b/src/gui/kernel/qstylehints.cpp index 69de74e2c3..9bd686070d 100644 --- a/src/gui/kernel/qstylehints.cpp +++ b/src/gui/kernel/qstylehints.cpp @@ -129,7 +129,8 @@ int QStyleHints::touchDoubleTapDistance() const */ Qt::Appearance QStyleHints::appearance() const { - return d_func()->appearance(); + Q_D(const QStyleHints); + return d->appearance(); } /*! @@ -597,10 +598,11 @@ int QStyleHints::mouseQuickSelectionThreshold() const */ void QStyleHintsPrivate::setAppearance(Qt::Appearance appearance) { - if (m_appearance != appearance) { - m_appearance = appearance; - emit q_func()->appearanceChanged(appearance); - } + if (m_appearance == appearance) + return; + m_appearance = appearance; + Q_Q(QStyleHints); + emit q->appearanceChanged(appearance); } QStyleHintsPrivate *QStyleHintsPrivate::get(QStyleHints *q) |