From a35f5682739142f826cd9cdb526c263645edac9b Mon Sep 17 00:00:00 2001 From: Adrien Destugues Date: Fri, 17 Jan 2014 14:45:11 +0100 Subject: [PATCH] Revert part of previous patch Checking only the visible resolution and colorspace isn't enough to call two modes equal, otherwise it is not possible to change the refresh rate anymore. --- src/add-ons/accelerants/intel_extreme/mode.cpp | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/src/add-ons/accelerants/intel_extreme/mode.cpp b/src/add-ons/accelerants/intel_extreme/mode.cpp index f484863fbc..c68aea21a1 100644 --- a/src/add-ons/accelerants/intel_extreme/mode.cpp +++ b/src/add-ons/accelerants/intel_extreme/mode.cpp @@ -789,12 +789,9 @@ intel_set_display_mode(display_mode* mode) uint32 colorMode, bytesPerRow, bitsPerPixel; get_color_space_format(target, colorMode, bytesPerRow, bitsPerPixel); - // avoid screen being off when switching workspaces when they have the same - // resolution. - if (target.virtual_width == display_mode_hook.mode.width - && target.virtual_height == display_mode_hook.mode.height - && target.space == display_mode_hook.mode.space) - return B_OK; + // TODO stop here, when the requested mode is the same as the current one. + // This would avoid screen flickering when setting a mode that's already in + // place. #if 0 static bool first = tru;