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

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>

1 files changed, 3 insertions, 1 deletions

diff --git a/src/gui/kernel/qplatformdialoghelper.h b/src/gui/kernel/qplatformdialoghelper.h
index 150a2333c1..1f8a7ec0a9 100644
--- a/src/gui/kernel/qplatformdialoghelper.h
+++ b/src/gui/kernel/qplatformdialoghelper.h
@@ -405,7 +405,9 @@ protected:
 
 public:
     // Keep in sync with QMessageBox Option
-    enum class Option : quint8 { DontUseNativeDialog = 0x00000001 };
+    enum class Option {
+        DontUseNativeDialog = 0x00000001,
+    };
     Q_DECLARE_FLAGS(Options, Option);
     Q_FLAG(Options);