cpp/openttd-patchpack/source Changeset - r25013:4c02a1340d93

Changeset - r25013:4c02a1340d93

Parent rev.

Child rev.

[Not reviewed]

master

0 2 0

Michael Lutz - 3 years ago 2021-03-08 12:48:32
michi@icosahedron.de

Codechange: don't set the window position when changing blitter

There really is no need to make an extra call to the OS in
these cases.

2 files changed with 6 insertions and 5 deletions:

src/video/win32_v.cpp

src/video/win32_v.h

0 comments (0 inline, 0 general)

src/video/win32_v.cpp

➞

Show inline comments

@@ @@ -128,9 +128,10 @@ uint8 VideoDriver_Win32Base::GetFullscre @@
 /**
  * Instantiate a new window.
  * @param full_screen Whether to make a full screen window or not.
  * @param resize Whether to change window size.
  * @return True if the window could be created.
  */
 bool VideoDriver_Win32Base::MakeWindow(bool full_screen)
+bool VideoDriver_Win32Base::MakeWindow(bool full_screen, bool resize)
+{
 	/* full_screen is whether the new window should be fullscreen,
 	 * _wnd.fullscreen is whether the current window is. */
@@ @@ -172,7 +173,7 @@ bool VideoDriver_Win32Base::MakeWindow(b @@
+		}
 		if (ChangeDisplaySettings(&settings, CDS_FULLSCREEN) != DISP_CHANGE_SUCCESSFUL) {
 			this->MakeWindow(false);  // don't care about the result
+			this->MakeWindow(false, resize);  // don't care about the result
 			return false;  // the request failed
+		}
 	} else if (this->fullscreen) {
@@ @@ -205,7 +206,7 @@ bool VideoDriver_Win32Base::MakeWindow(b @@
 		h = r.bottom - r.top;
 		if (this->main_wnd != nullptr) {
 			if (!_window_maximize) SetWindowPos(this->main_wnd, 0, 0, 0, w, h, SWP_NOACTIVATE | SWP_NOOWNERZORDER | SWP_NOZORDER | SWP_NOMOVE);
+			if (!_window_maximize && resize) SetWindowPos(this->main_wnd, 0, 0, 0, w, h, SWP_NOACTIVATE | SWP_NOOWNERZORDER | SWP_NOZORDER | SWP_NOMOVE);
 		} else {
 			int x = (GetSystemMetrics(SM_CXSCREEN) - w) / 2;
 			int y = (GetSystemMetrics(SM_CYSCREEN) - h) / 2;
@@ @@ -1043,7 +1044,7 @@ bool VideoDriver_Win32GDI::AllocateBacki @@
 bool VideoDriver_Win32GDI::AfterBlitterChange()
+{
 	assert(BlitterFactory::GetCurrentBlitter()->GetScreenDepth() != 0);
 	return this->AllocateBackingStore(_screen.width, _screen.height, true) && this->MakeWindow(_fullscreen);
+	return this->AllocateBackingStore(_screen.width, _screen.height, true) && this->MakeWindow(_fullscreen, false);
+}
 void VideoDriver_Win32GDI::MakePalette()

src/video/win32_v.h

➞

Show inline comments

@@ @@ -54,7 +54,7 @@ protected: @@
 	bool PollEvent() override;
 	void Initialize();
 	bool MakeWindow(bool full_screen);
+	bool MakeWindow(bool full_screen, bool resize = true);
 	void ClientSizeChanged(int w, int h, bool force = false);
 	/** Get screen depth to use for fullscreen mode. */

0 comments (0 inline, 0 general)