Skip to content

Mt blocking#2

Open
LevKats wants to merge 7 commits intomasterfrom
mt_blocking
Open

Mt blocking#2
LevKats wants to merge 7 commits intomasterfrom
mt_blocking

Conversation

@LevKats
Copy link
Copy Markdown
Owner

@LevKats LevKats commented Mar 26, 2020

No description provided.

Copy link
Copy Markdown

@xphoenix xphoenix left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Клиентские потоки могут висеть в read или в send.
Во время остановки сервера нужно прервать все read операции, т.е сделать shutdown(socket, SHUT_RD)

void ServerImpl::Join() {
assert(_thread.joinable());
_thread.join();
close(_server_socket);
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Это должен делать поток accept

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.

2 participants