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

Use namespace std::experimental::net where possible #65

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

LarsGullik
Copy link
Contributor

Collapse

namespace std {
namespace experimental {
namespace net
}}}

to

namespace std::experimental::net {
}

@jwakely
Copy link
Member

jwakely commented Mar 30, 2017

That's a C++17 feature, so we can't use it in the TS which is based on C++14.

In theory we could rebase it on C++17, so if I get time I might write a paper proposing that for Toronto.

@LarsGullik
Copy link
Contributor Author

LarsGullik commented Mar 30, 2017 via email

@jwakely
Copy link
Member

jwakely commented Mar 30, 2017

There's an NB comment about the normative references, and the resolution for that adds a reference to ISO/IEC 14882:2014, but it can be changed. I'll accept this change if/when we rebase it, but it's arguably not editorial now.

Collapse

namespace std {
namespace experimental {
namespace net
}}}

to

namespace std::experimental::net {
}
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