Skip to content

fix(Thread): fix Thread reuse error, add thread interrupt feature #4942

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 27 commits into
base: main
Choose a base branch
from

Conversation

siren186
Copy link
Member

  1. fix Thread reuse error
Poco::Thread thread;
thread.start(runnable);
thread.join();

// reuse error
thread.start(runable);
thread.join(); // Error !!!
  1. add thread interrupt feature

See:

Poco demo:

Poco::Thread thread;
thread.interrupt();

// way-1:
bool ret = Poco::Thread::current()->isInterrupted();

// way-2:
Poco::Thread::current()->checkInterrupted(); // throw Poco::ThreadInterruptedException

aleks-f and others added 27 commits March 20, 2025 12:34
* fix(VarHolder): MSVC warning pocoproject#4886

* fix(VarHolder): do not convert unsigned pocoproject#4886

* chore(SocketImpl): eliminate MSVC warning

* fix(VarHolder): do not convert unsigned (extract unpreserveSign()) pocoproject#4886

* chore(testsuite): eliminate MSVC warnings

* chore(VarHolder): simplify unpreserveSign using constexpr.

---------

Co-authored-by: Matej Kenda <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants