### Problem description This PR addresses issue #2013 that described a cluttered Event Manager. This is a DX issue and should not impact the users whatsoever. ### Implementation description The changes revolve around three main points: 1. the Event Manager (`event_manager.hpp`) was split into four categories: GUI, Interaction, Lifecycle, and Provider, and two types: Events, and Requests. This results in the following files: - `events_gui.hpp` - `events_interaction.hpp` - `events_lifecycle.hpp` - `events_provider.hpp` - `requests_gui.hpp` - `requests_interaction.hpp` - `requests_lifecycle.hpp` - `requests_provider.hpp` 2. Every event and request now has its own piece of documentation, with a `@brief`, accompanied by a longer comment if needed, and gets its `@param`s described. 3. The old `event_manager.hpp` import was removed and replaced by the correct imports wherever needed, as to reduce spread of those files only to where they are truly useful. ### Additional things The commits have been split into (chrono-)logical steps: - `feat`: split the Event Manager, and replace the imports - `refactor`, `chore`: make various small changes to match the required structure - `docs`: add documentation for events and requests Hopefully, this will help to review the PR. *Note: Beware of very long rebuild times in between the commits, use them sparingly! The Actions will ensure this PR builds anyways* Closes #2013 --------- Signed-off-by: BioTheWolff <47079795+BioTheWolff@users.noreply.github.com> Co-authored-by: Nik <werwolv98@gmail.com>
88 lines
1.8 KiB
C++
88 lines
1.8 KiB
C++
#pragma once
|
|
|
|
#include <hex/api/imhex_api.hpp>
|
|
|
|
namespace hex {
|
|
|
|
namespace impl {
|
|
|
|
class AutoResetBase {
|
|
public:
|
|
virtual ~AutoResetBase() = default;
|
|
virtual void reset() = 0;
|
|
};
|
|
|
|
}
|
|
|
|
template<typename T>
|
|
class AutoReset : public impl::AutoResetBase {
|
|
public:
|
|
using Type = T;
|
|
|
|
AutoReset() {
|
|
ImHexApi::System::impl::addAutoResetObject(this);
|
|
}
|
|
|
|
T* operator->() {
|
|
return &m_value;
|
|
}
|
|
|
|
const T* operator->() const {
|
|
return &m_value;
|
|
}
|
|
|
|
T& operator*() {
|
|
return m_value;
|
|
}
|
|
|
|
const T& operator*() const {
|
|
return m_value;
|
|
}
|
|
|
|
operator T&() {
|
|
return m_value;
|
|
}
|
|
|
|
operator const T&() const {
|
|
return m_value;
|
|
}
|
|
|
|
T& operator=(const T &value) {
|
|
m_value = value;
|
|
m_valid = true;
|
|
return m_value;
|
|
}
|
|
|
|
T& operator=(T &&value) noexcept {
|
|
m_value = std::move(value);
|
|
m_valid = true;
|
|
return m_value;
|
|
}
|
|
|
|
bool isValid() const {
|
|
return m_valid;
|
|
}
|
|
|
|
private:
|
|
friend void ImHexApi::System::impl::cleanup();
|
|
|
|
void reset() override {
|
|
if constexpr (requires { m_value.reset(); }) {
|
|
m_value.reset();
|
|
} else if constexpr (requires { m_value.clear(); }) {
|
|
m_value.clear();
|
|
} else if constexpr (requires(T t) { t = nullptr; }) {
|
|
m_value = nullptr;
|
|
} else {
|
|
m_value = { };
|
|
}
|
|
|
|
m_valid = false;
|
|
}
|
|
|
|
private:
|
|
bool m_valid = true;
|
|
T m_value;
|
|
};
|
|
|
|
} |