fix(NotificationQueue): wakeUpAll() lost-wakeup race causes deadlock …

[aleks-f]

…on shutdown #5213

[Copilot]

Pull request overview

Fixes a shutdown-time deadlock in NotificationQueue by preventing a lost-wakeup race when wakeUpAll() happens before threads enter waitDequeueNotification().

Changes:

• Add _wokeUp state so waitDequeueNotification() can return immediately if a prior wakeUpAll() occurred.
• Add a new regression test that reproduces the “wakeUpAll before wait” scenario.
• Register the new test in the test suite.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.

File	Description
Foundation/testsuite/src/NotificationQueueTest.h	Declares the new regression test.
Foundation/testsuite/src/NotificationQueueTest.cpp	Implements and registers the regression test for the lost-wakeup scenario.
Foundation/src/NotificationQueue.cpp	Makes waitDequeueNotification*() return nullptr if _wokeUp was set, and sets _wokeUp in wakeUpAll().
Foundation/include/Poco/NotificationQueue.h	Adds _wokeUp member to persist wakeup state across calls.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

std::move is used later in this file, but <utility> is not included. Please add #include <utility> to ensure the file compiles portably without relying on transitive includes.

Suggested change

	#include <utility>