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
How to space ref qualifiers #5308
Comments
This seems prevalent, so I've been suggesting removing spaces whenever I notice them. |
I'm mildly in favor of having a space there. |
Yeah, I see what you mean. I don't feel strongly either way, but do you at least think we should fix a style at all? |
Yes, there should be consistent style enforced with check-source.sh (if that is technically feasible). |
OK, great! I leave the choice to you -- I think I can see the point of cv and ref being different, separate things here, esp. since we also have a space before a |
I generally agree: I don't tend to think of those qualifiers as actually being decl-specifiers and declarator operators, perhaps because ref-qualifiers don't influence the type of |
Jens: wants space, "too close together without space" OpenSDH: wants space, becausethe qualifiers aren't decl-specifiers Richard: wants space, because it's "const [T]&" where the T s omitted. => Let's go with the space then! |
spacey! |
Should we have a convention for how to use whitespace in ref qualifiers in the library?
These qualifiers are still relatively rare, so we don't really have a weight of precedence here. I'm inclined to have no space between the cv and the ref qualifier (
const&
). If we want to fix a style, we should record it in the wiki.The text was updated successfully, but these errors were encountered: