dave/openttd
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix: thread safety issue during exiting the game (#9380)

Browse Source 
_exit_game is read by the draw-thread to know when to exit, but
most of the time written by the game-thread.
This commit is contained in:
Patric Stout
2021-06-17 18:58:59 +02:00
committed by GitHub
parent 281a65b3e1
commit c12a152ec9
2 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/gfx.cpp
View File

@@ -41,7 +41,7 @@ bool _right_button_down; ///< Is right mouse button pressed?
bool _right_button_clicked; ///< Is right mouse button clicked? bool _right_button_clicked; ///< Is right mouse button clicked?
DrawPixelInfo _screen; DrawPixelInfo _screen;
bool _screen_disable_anim = false; ///< Disable palette animation (important for 32bpp-anim blitter during giant screenshot) bool _screen_disable_anim = false; ///< Disable palette animation (important for 32bpp-anim blitter during giant screenshot)
bool _exit_game; std::atomic<bool> _exit_game;
GameMode _game_mode; GameMode _game_mode;
SwitchMode _switch_mode; ///< The next mainloop command. SwitchMode _switch_mode; ///< The next mainloop command.
PauseMode _pause_mode; PauseMode _pause_mode;

3
src/openttd.h
View File

@@ -10,6 +10,7 @@
#ifndef OPENTTD_H #ifndef OPENTTD_H
#define OPENTTD_H #define OPENTTD_H
#include <atomic>
#include "core/enum_type.hpp" #include "core/enum_type.hpp"
/** Mode which defines the state of the game. */ /** Mode which defines the state of the game. */
@@ -52,7 +53,7 @@ enum DisplayOptions {
extern GameMode _game_mode; extern GameMode _game_mode;
extern SwitchMode _switch_mode; extern SwitchMode _switch_mode;
extern bool _exit_game; extern std::atomic<bool> _exit_game;
extern bool _save_config; extern bool _save_config;
/** Modes of pausing we've got */ /** Modes of pausing we've got */