mirrors/SFML
1
0
Fork 0
mirror of https://github.com/SFML/SFML.git synced 2025-01-31 13:45:13 +08:00
Code Issues Packages Projects Releases Wiki Activity

Add non-const overload of Event::visit

Browse Source
This commit is contained in:
vittorioromeo 2025-01-20 16:24:04 +01:00 committed by Vittorio Romeo
parent 0e37e6dfee
commit 18393ea5cb
4 changed files with 58 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
include/SFML/Window/Event.hpp
View File

@ -338,6 +338,17 @@ public:
template <typename TEventSubtype>
[[nodiscard]] const TEventSubtype* getIf() const;
////////////////////////////////////////////////////////////
/// \brief Apply a visitor to the event
///
/// \param visitor The visitor to apply
///
/// \return The result of applying the visitor to the event
///
////////////////////////////////////////////////////////////
template <typename T>
decltype(auto) visit(T&& visitor);
////////////////////////////////////////////////////////////
/// \brief Apply a visitor to the event
///

8
include/SFML/Window/Event.inl
View File

@ -78,6 +78,14 @@ const TEventSubtype* Event::getIf() const
}
////////////////////////////////////////////////////////////
template <typename T>
decltype(auto) Event::visit(T&& visitor)
{
return std::visit(std::forward<T>(visitor), m_data);
}
////////////////////////////////////////////////////////////
template <typename T>
decltype(auto) Event::visit(T&& visitor) const

2
include/SFML/Window/WindowBase.inl
View File

@ -68,7 +68,7 @@ void WindowBase::handleEvents(Ts&&... handlers) // NOLINT(cppcoreguidelines-miss
// NOLINTNEXTLINE(misc-const-correctness)
priv::OverloadSet overloadSet{std::forward<Ts>(handlers)..., [](const priv::DelayOverloadResolution&) { /* ignore */ }};
while (const std::optional event = pollEvent())
while (std::optional event = pollEvent())
event->visit(overloadSet);
}

47
test/Window/Event.test.cpp
View File

@ -9,17 +9,22 @@ namespace
{
struct
{
std::string_view operator()(const sf::Event::Closed&) const
std::string_view operator()(sf::Event::Closed&) const
{
return "Closed";
return "Closed (non-const)";
}
std::string_view operator()(const sf::Event::Resized&) const
std::string_view operator()(const sf::Event::Closed&) const
{
return "Closed (const)";
}
std::string_view operator()(sf::Event::Resized&) const
{
return "Resized";
}
std::string_view operator()(const sf::Event::KeyPressed&) const
std::string_view operator()(sf::Event::KeyPressed) const
{
return "KeyPressed";
}
@ -306,10 +311,34 @@ TEST_CASE("[Window] sf::Event")
SECTION("visit()")
{
CHECK(sf::Event(sf::Event::Closed{}).visit(visitor) == "Closed");
CHECK(sf::Event(sf::Event::Resized{}).visit(visitor) == "Resized");
CHECK(sf::Event(sf::Event::FocusLost{}).visit(visitor) == "Other");
CHECK(sf::Event(sf::Event::FocusGained{}).visit(visitor) == "Other");
CHECK(sf::Event(sf::Event::KeyPressed{}).visit(visitor) == "KeyPressed");
SECTION("Non-const")
{
sf::Event closed = sf::Event::Closed{};
CHECK(closed.visit(visitor) == "Closed (non-const)");
sf::Event resized = sf::Event::Resized{};
CHECK(resized.visit(visitor) == "Resized");
sf::Event keyPressed = sf::Event::KeyPressed{};
CHECK(keyPressed.visit(visitor) == "KeyPressed");
sf::Event focusLost = sf::Event::FocusLost{};
CHECK(focusLost.visit(visitor) == "Other");
}
SECTION("Const")
{
const sf::Event closed = sf::Event::Closed{};
CHECK(closed.visit(visitor) == "Closed (const)");
const sf::Event resized = sf::Event::Resized{};
CHECK(resized.visit(visitor) == "Other"); // Cannot use non-const reference callback
const sf::Event keyPressed = sf::Event::KeyPressed{};
CHECK(keyPressed.visit(visitor) == "KeyPressed");
const sf::Event focusLost = sf::Event::FocusLost{};
CHECK(focusLost.visit(visitor) == "Other");
}
}
}