AuroraMiddleware/qt5base-lts
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

QMessageBox: leave the Option's underlying_type as the default, int

Browse Source 
QFlags<Option> will anyway store the value in an `int`, so we gain
nothing from choosing quint8 as the fixed type, except artificially
limiting the number of possible future options to eight.

Found in API-review.

As a drive-by, put options one per line and add a trailing comma where
possible.

Pick-to: 6.6
Change-Id: I0ed588ea6c7912e50705af89c47e2cf7e2ee23a0
Reviewed-by: Volker Hilsheimer <volker.hilsheimer@qt.io>
This commit is contained in:
Marc Mutz 2023-08-11 22:21:51 +02:00
parent 5c7e600b41
commit d8952690c7
2 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/gui/kernel/qplatformdialoghelper.h
View File

@ -405,7 +405,9 @@ protected:
public: public:
// Keep in sync with QMessageBox Option // Keep in sync with QMessageBox Option
enum class Option : quint8 { DontUseNativeDialog = 0x00000001 }; enum class Option {
DontUseNativeDialog = 0x00000001,
};
Q_DECLARE_FLAGS(Options, Option); Q_DECLARE_FLAGS(Options, Option);
Q_FLAG(Options); Q_FLAG(Options);

2
src/widgets/dialogs/qmessagebox.h
View File

@ -34,7 +34,7 @@ class Q_WIDGETS_EXPORT QMessageBox : public QDialog
Q_PROPERTY(Options options READ options WRITE setOptions) Q_PROPERTY(Options options READ options WRITE setOptions)
public: public:
// Keep in sync with MessageBoxOption in qplatformdialoghelper.h // Keep in sync with MessageBoxOption in qplatformdialoghelper.h
enum class Option : quint8 { enum class Option {
DontUseNativeDialog = 0x00000001 DontUseNativeDialog = 0x00000001
}; };
Q_FLAG(Option) Q_FLAG(Option)