Skip to content
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

boost::asio::io_service -> boost::asio::io_context #80

Merged
merged 1 commit into from
May 30, 2022

Conversation

adamncasey
Copy link
Contributor

io_service is a deprecated name, I think asio renamed this as part of aligning with the networking TS (possibly coming in C++23? who knows). This likely isn't the only thing which changed, but it stands out when reading the codebase. https://www.boost.org/doc/libs/1_79_0/doc/html/boost_asio/net_ts.html is a reference if anyone's interested

@Chinmay1412
Copy link
Contributor

I guess, we can change the variable name from ioService to ioContext. But that will not prevent us for merging this PR.

@adamncasey adamncasey force-pushed the iosioc branch 2 times, most recently from fb9c977 to 3bde9d8 Compare May 27, 2022 13:51
I think asio renamed this as part of aligning with the networking TS

This might also help us move to that if it's ever standardised.
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