diff options
author | Volker Hilsheimer <volker.hilsheimer@qt.io> | 2020-07-02 16:50:24 +0200 |
---|---|---|
committer | Volker Hilsheimer <volker.hilsheimer@qt.io> | 2020-07-04 02:51:35 +0200 |
commit | a10bf2ec5a3f8a7c7e66e6c288ca0ca85043d5d1 (patch) | |
tree | 43a298c1f124344720e7a85e060a73ba2c31b166 /src/widgets/widgets/qplaintextedit_p.h | |
parent | ae890390e523f0f435190654b081764df88d7b94 (diff) |
Improve data member ordering and initialization in some private classes
Put bit fields together, and generally order by size to maximize the
compiler's ability to pack data. Initialize in declaration where
possible.
Change-Id: I49d4a3d2271229d671aeaf3eb17e6b7ecdc5099e
Reviewed-by: Shawn Rutledge <shawn.rutledge@qt.io>
Diffstat (limited to 'src/widgets/widgets/qplaintextedit_p.h')
-rw-r--r-- | src/widgets/widgets/qplaintextedit_p.h | 35 |
1 files changed, 14 insertions, 21 deletions
diff --git a/src/widgets/widgets/qplaintextedit_p.h b/src/widgets/widgets/qplaintextedit_p.h index 6d82cabb93..112eb2e4f6 100644 --- a/src/widgets/widgets/qplaintextedit_p.h +++ b/src/widgets/widgets/qplaintextedit_p.h @@ -143,26 +143,30 @@ public: void updateDefaultTextOption(); - QPlainTextEditControl *control; - - bool tabChangesFocus; - QBasicTimer autoScrollTimer; +#ifdef QT_KEYPAD_NAVIGATION + QBasicTimer deleteAllTimer; +#endif QPoint autoScrollDragPos; + QString placeholderText; - QPlainTextEdit::LineWrapMode lineWrap; - QTextOption::WrapMode wordWrap; - Qt::KeyboardModifiers keyboardModifiers; + QPlainTextEditControl *control = nullptr; + qreal topLineFracture = 0; // for non-int sized fonts + qreal pageUpDownLastCursorY = 0; + QPlainTextEdit::LineWrapMode lineWrap = QPlainTextEdit::WidgetWidth; + QTextOption::WrapMode wordWrap = QTextOption::WrapAtWordBoundaryOrAnywhere; + Qt::KeyboardModifiers keyboardModifiers = {}; + int originalOffsetY = 0; + int topLine = 0; + uint tabChangesFocus : 1; uint showCursorOnInitialShow : 1; uint backgroundVisible : 1; uint centerOnScroll : 1; uint inDrag : 1; uint clickCausedFocus : 1; uint placeholderVisible : 1; - - int topLine; - qreal topLineFracture; // for non-int sized fonts + uint pageUpDownLastCursorYIsValid : 1; void setTopLine(int visualTopLine, int dx = 0); void setTopBlock(int newTopBlock, int newTopLine, int dx = 0); @@ -175,19 +179,8 @@ public: void append(const QString &text, Qt::TextFormat format = Qt::AutoText); - qreal pageUpDownLastCursorY; - bool pageUpDownLastCursorYIsValid; - - -#ifdef QT_KEYPAD_NAVIGATION - QBasicTimer deleteAllTimer; -#endif - void _q_cursorPositionChanged(); void _q_modificationChanged(bool); - - int originalOffsetY; - QString placeholderText; }; QT_END_NAMESPACE |