Browse Source

LibGUI: Use clamp() is various places

Andreas Kling 5 years ago
parent
commit
b25210ee1b

+ 1 - 4
Libraries/LibGUI/GProgressBar.cpp

@@ -53,10 +53,7 @@ void GProgressBar::set_range(int min, int max)
     ASSERT(min < max);
     m_min = min;
     m_max = max;
-    if (m_value > m_max)
-        m_value = m_max;
-    if (m_value < m_min)
-        m_value = m_min;
+    m_value = clamp(m_value, m_min, m_max);
 }
 
 void GProgressBar::paint_event(GPaintEvent& event)

+ 2 - 8
Libraries/LibGUI/GScrollBar.cpp

@@ -129,10 +129,7 @@ void GScrollBar::set_range(int min, int max)
     m_max = max;
 
     int old_value = m_value;
-    if (m_value < m_min)
-        m_value = m_min;
-    if (m_value > m_max)
-        m_value = m_max;
+    m_value = clamp(m_value, m_min, m_max);
     if (on_change && m_value != old_value)
         on_change(m_value);
 
@@ -141,10 +138,7 @@ void GScrollBar::set_range(int min, int max)
 
 void GScrollBar::set_value(int value)
 {
-    if (value < m_min)
-        value = m_min;
-    if (value > m_max)
-        value = m_max;
+    value = clamp(value, m_min, m_max);
     if (value == m_value)
         return;
     m_value = value;

+ 2 - 9
Libraries/LibGUI/GSlider.cpp

@@ -50,20 +50,13 @@ void GSlider::set_range(int min, int max)
         return;
     m_min = min;
     m_max = max;
-
-    if (m_value > max)
-        m_value = max;
-    if (m_value < min)
-        m_value = min;
+    m_value = clamp(m_value, m_min, m_max);
     update();
 }
 
 void GSlider::set_value(int value)
 {
-    if (value > m_max)
-        value = m_max;
-    if (value < m_min)
-        value = m_min;
+    value = clamp(value, m_min, m_max);
     if (m_value == value)
         return;
     m_value = value;

+ 2 - 8
Libraries/LibGUI/GSpinBox.cpp

@@ -59,10 +59,7 @@ GSpinBox::~GSpinBox()
 
 void GSpinBox::set_value(int value)
 {
-    if (value < m_min)
-        value = m_min;
-    if (value > m_max)
-        value = m_max;
+    value = clamp(value, m_min, m_max);
     if (m_value == value)
         return;
     m_value = value;
@@ -82,10 +79,7 @@ void GSpinBox::set_range(int min, int max)
     m_max = max;
 
     int old_value = m_value;
-    if (m_value < m_min)
-        m_value = m_min;
-    if (m_value > m_max)
-        m_value = m_max;
+    m_value = clamp(m_value, m_min, m_max);
     if (on_change && m_value != old_value)
         on_change(m_value);