Files
@ r24498:e9114d9ab04a
Branch filter:
Location: cpp/openttd-patchpack/source/src/thread.h - annotation
r24498:e9114d9ab04a
2.4 KiB
text/x-c
Fix #6468: don't store version of AIs-started-via-console in name
You can do: "startai myai.3", which starts version 3 of "myai".
This is very useful for testing save/load code between different
versions of your AI.
However, when using this syntax, the AI got saved as "myai.3" as
name of the AI, instead of "myai". This caused several problems,
like indicating to the user the AI could not be found, but still
load the AI. But in all cases, the AI never got the chance to
load the saved data, making the whole reason this exists pointless.
By splitting the name and version already in the console command,
the code becomes simpler and AIs started this way now follow the
normal flow after initialization.
You can do: "startai myai.3", which starts version 3 of "myai".
This is very useful for testing save/load code between different
versions of your AI.
However, when using this syntax, the AI got saved as "myai.3" as
name of the AI, instead of "myai". This caused several problems,
like indicating to the user the AI could not be found, but still
load the AI. But in all cases, the AI never got the chance to
load the saved data, making the whole reason this exists pointless.
By splitting the name and version already in the console command,
the code becomes simpler and AIs started this way now follow the
normal flow after initialization.
r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23589:b3521e885d81 r23589:b3521e885d81 r23589:b3521e885d81 r23589:b3521e885d81 r23589:b3521e885d81 r23589:b3521e885d81 r23589:b3521e885d81 r23589:b3521e885d81 r23589:b3521e885d81 r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23607:36c15679007d r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23607:36c15679007d r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec r23587:821735a25dec | /*
* This file is part of OpenTTD.
* OpenTTD is free software; you can redistribute it and/or modify it under the terms of the GNU General Public License as published by the Free Software Foundation, version 2.
* OpenTTD is distributed in the hope that it will be useful, but WITHOUT ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
* See the GNU General Public License for more details. You should have received a copy of the GNU General Public License along with OpenTTD. If not, see <http://www.gnu.org/licenses/>.
*/
/** @file thread.h Base of all threads. */
#ifndef THREAD_H
#define THREAD_H
#include "debug.h"
#include <system_error>
#include <thread>
/** Signal used for signalling we knowingly want to end the thread. */
class OTTDThreadExitSignal { };
/**
* Sleep on the current thread for a defined time.
* @param milliseconds Time to sleep for in milliseconds.
*/
inline void CSleep(int milliseconds)
{
std::this_thread::sleep_for(std::chrono::milliseconds(milliseconds));
}
/**
* Name the thread this function is called on for the debugger.
* @param name Name to set for the thread..
*/
void SetCurrentThreadName(const char *name);
/**
* Start a new thread.
* @tparam TFn Type of the function to call on the thread.
* @tparam TArgs Type of the parameters of the thread function.
* @param thr Pointer to a thread object; may be \c nullptr if a detached thread is wanted.
* @param name Name of the thread.
* @param _Fx Function to call on the thread.
* @param _Ax Arguments for the thread function.
* @return True if the thread was successfully started, false otherwise.
*/
template<class TFn, class... TArgs>
inline bool StartNewThread(std::thread *thr, const char *name, TFn&& _Fx, TArgs&&... _Ax)
{
#ifndef NO_THREADS
try {
std::thread t([] (const char *name, TFn&& F, TArgs&&... A) {
SetCurrentThreadName(name);
try {
/* Call user function with the given arguments. */
F(A...);
} catch (OTTDThreadExitSignal&) {
} catch (...) {
NOT_REACHED();
}
}, name, std::forward<TFn>(_Fx), std::forward<TArgs>(_Ax)...);
if (thr != nullptr) {
*thr = std::move(t);
} else {
t.detach();
}
return true;
} catch (const std::system_error& e) {
/* Something went wrong, the system we are running on might not support threads. */
DEBUG(misc, 1, "Can't create thread '%s': %s", name, e.what());
}
#endif
return false;
}
#endif /* THREAD_H */
|