Removed Qt 5 todo comments from qpalette.h
authorSamuel Rødal <samuel.rodal@nokia.com>
Mon, 2 Apr 2012 07:28:49 +0000 (09:28 +0200)
committerQt by Nokia <qt-info@nokia.com>
Wed, 4 Apr 2012 07:00:24 +0000 (09:00 +0200)
The removal won't happen in Qt 5, and there's no great value in making
NoRole have the value 0 at the potential risk of breaking existing code.

Task-number: QTBUG-25068
Change-Id: I57238f55a23e77a783ebac47b96b4a5e62480add
Reviewed-by: João Abecasis <joao.abecasis@nokia.com>
src/gui/kernel/qpalette.h

index ca619a8..96295ab 100644 (file)
@@ -86,12 +86,12 @@ public:
     enum ColorRole { WindowText, Button, Light, Midlight, Dark, Mid,
                      Text, BrightText, ButtonText, Base, Window, Shadow,
                      Highlight, HighlightedText,
-                     Link, LinkVisited, // ### Qt 5: remove
+                     Link, LinkVisited,
                      AlternateBase,
-                     NoRole, // ### Qt 5: value should be 0 or -1
+                     NoRole,
                      ToolTipBase, ToolTipText,
                      NColorRoles = ToolTipText + 1,
-                     Foreground = WindowText, Background = Window // ### Qt 5: remove
+                     Foreground = WindowText, Background = Window
                    };
 
     inline ColorGroup currentColorGroup() const { return static_cast<ColorGroup>(current_group); }