Match against current rate if no request for a specific rate was made

[libs/gui.git] / source / graphics / display.cpp

diff --git a/source/graphics/display.cpp b/source/graphics/display.cpp
index 1c7b639a06d8f993b72ef92fb19d9d211271ed5b..7f0b25af22f77d177ef244daa3997bc809b7951f 100644 (file)
--- a/source/graphics/display.cpp
+++ b/source/graphics/display.cpp
@@ -25,42 +25,49 @@ void Display::remove_window(Window &wnd)
 
 const VideoMode &Display::get_desktop_mode() const
 {
-       if(!primary_monitor || !primary_monitor->desktop_mode)
+       if(!primary_monitor || !primary_monitor->desktop_settings.mode)
                throw logic_error("no desktop mode");
-       return *primary_monitor->desktop_mode;
+       return *primary_monitor->desktop_settings.mode;
 }
 
 void Display::restore_mode()
 {
        for(list<Monitor>::const_iterator i=monitors.begin(); i!=monitors.end(); ++i)
-               if(i->desktop_mode)
-                       set_mode(*i->desktop_mode, false);
+               if(i->desktop_settings.mode)
+                       set_mode(*i->desktop_settings.mode, false);
 }
 
 const VideoMode *Display::find_mode(const VideoMode &mode, float rate_tolerance) const
 {
        const VideoMode *closest = 0;
-       float rate_diff = mode.rate;
+       float rate_diff = 0.0f;
+
+       float target_rate = mode.rate;
+       if(!target_rate && mode.monitor)
+               if(const VideoMode *current_mode = mode.monitor->current_settings.mode)
+                       target_rate = current_mode->rate;
+
        for(list<VideoMode>::const_iterator i=modes.begin(); i!=modes.end(); ++i)
        {
                if(mode.monitor && i->monitor!=mode.monitor)
                        continue;
+               // Only check rate tolerance against an explicitly requested rate
                if(mode.rate && (i->rate<mode.rate-rate_tolerance || i->rate>mode.rate+rate_tolerance))
                        continue;
                if(i->width!=mode.width || i->height!=mode.height)
                        continue;
 
-               if(mode.rate)
+               float ref_rate = target_rate;
+               if(!ref_rate)
+                       if(const VideoMode *current_mode = i->monitor->current_settings.mode)
+                               ref_rate = current_mode->rate;
+
+               float d = abs(i->rate-ref_rate);
+               if(!closest || d<rate_diff)
                {
-                       float d = abs(i->rate-mode.rate);
-                       if(d<rate_diff)
-                       {
-                               closest = &*i;
-                               rate_diff = d;
-                       }
+                       closest = &*i;
+                       rate_diff = d;
                }
-               else
-                       return &*i;
        }
 
        return closest;