-
-
Notifications
You must be signed in to change notification settings - Fork 4
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
posix namespace is reserved #1
Comments
Thanks. We are indeed aware of the issue, and that's the primary reason we aren't promoting this project beyond its existing user base where the matter is of little concern (since compilers don't enforce §17.6.4.2.2, at present). |
I don't get this impression from the readme/welcome page at all:
I'd say usage outside a closed group is promoted. |
You'd have to find your way to the README first. Based on the number of people who've starred the project over the past few years it's been up here, that doesn't happen very often. (Where did you guys come from, all of a sudden?) But if it makes someone feel better, we will go ahead and add a warning about §17.6.4.2.2 to the README. |
I appreciate you being open. I can see the surprise. Then again, it's a public project and you made that readme with a purpose :) So, I assumed visitors are welcome. I'm active on stackoverflow and the project was linked in a chat thread. Remember that if people come to talk to you about your project it means they care enough. I don't think we're trying to annoy you :) Do you prefer I make pull-requests instead of just "dropping" ideas? |
I've gone ahead and added a prominent "Caveats" section to the README just now. @sehe The README is based on a template common to all the projects around this corner of GitHub. The intent was originally to publish this project on SourceForge and elsewhere, as it certainly fills a niche of sorts and could be of use to many folks--especially given its public-domain licensing--but noticing §17.6.4.2.2 rather unmotivated me to pursue that plan of action. The project namespace probably does need to get renamed at some point, but it hasn't been a pressing issue (no compiler complains as yet) and it'd be a nasty bit of breakage to the library and the existing set of programs that use it, so it simply hasn't become a priority given no external users as yet. I welcome pull requests, of course, but something of the magnitude of changing the project namespace merits a discussion and plan before anyone starts changing hundreds of lines of existing code. |
That's what we were doing here, until the plan was closed :) Anyhoops. Thanks for the library. I will look at it over the coming period of time. Cheers |
OK, issue now reopened for continued collaborative input on the matter--and thanks for your interest. |
Maybe it is worth to actually go ahead and rename the namespace? |
According to the C++ standard, the
posix
namespace is reserved for future use and thus its use is illegal.Section in C++11 standard:
§17.6.4.2.2 [namespace.posix]
The text was updated successfully, but these errors were encountered: