cpp/openttd-patchpack/source Changeset - r24744:6381326705ae

Changeset - r24744:6381326705ae

Parent rev.

Child rev.

[Not reviewed]

master

0 1 0

Patric Stout - 4 years ago 2021-02-11 10:19:05
truebrain@openttd.org

Codechange: [SDL2] Don't use globals if we can do with locals

1 file changed with 4 insertions and 6 deletions:

src/video/sdl2_v.cpp

0 comments (0 inline, 0 general)

src/video/sdl2_v.cpp

➞

Show inline comments

@@ @@ -58,10 +58,6 @@ static bool _cursor_new_in_window = fals @@
 static SDL_Rect _dirty_rects[MAX_DIRTY_RECTS];
 static int _num_dirty_rects;
 /* Size of window */
 static int _window_size_w;
 static int _window_size_h;
 void VideoDriver_SDL::MakeDirty(int left, int top, int width, int height)
+{
 	if (_num_dirty_rects < MAX_DIRTY_RECTS) {
@@ @@ -929,9 +925,11 @@ bool VideoDriver_SDL::ToggleFullscreen(b @@
 	std::unique_lock<std::recursive_mutex> lock;
 	if (_draw_mutex != nullptr) lock = std::unique_lock<std::recursive_mutex>(*_draw_mutex);
 	int w, h;
 	/* Remember current window size */
 	if (fullscreen) {
-		SDL_GetWindowSize(_sdl_window, &_window_size_w, &_window_size_h);
+		SDL_GetWindowSize(_sdl_window, &w, &h);
 		/* Find fullscreen window size */
 		SDL_DisplayMode dm;
@@ @@ -947,7 +945,7 @@ bool VideoDriver_SDL::ToggleFullscreen(b @@
 	if (ret == 0) {
 		/* Switching resolution succeeded, set fullscreen value of window. */
 		_fullscreen = fullscreen;
-		if (!fullscreen) SDL_SetWindowSize(_sdl_window, _window_size_w, _window_size_h);
+		if (!fullscreen) SDL_SetWindowSize(_sdl_window, w, h);
 	} else {
 		DEBUG(driver, 0, "SDL_SetWindowFullscreen() failed: %s", SDL_GetError());
+	}

0 comments (0 inline, 0 general)