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.

Sign up for GitHub

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

Jump to bottom

[optional.syn] const placed on right #127

Closed
sdutoit opened this issue May 15, 2013 · 5 comments
Closed

[optional.syn] const placed on right #127

sdutoit opened this issue May 15, 2013 · 5 comments
Milestone
C++14 DIS nice-to...

Comments

@sdutoit
Copy link
Contributor

sdutoit commented May 15, 2013

Alisdair says:

// 20.6.4.5, observers
Pointers and references are placing 'const' on the right,
but standard is fairly consistently 'const' on the left.
(even though I prefer on-the-right myself!)
@sdutoit
Copy link
Contributor Author

sdutoit commented Aug 29, 2013

I started by fixing this for pointers throughout the draft in ce29631 - references are next, and the fixes won't be limited to just this section.

@sdutoit
Copy link
Contributor Author

sdutoit commented Aug 29, 2013

Err; technically the last comment was only related to this and not exactly the same, but it's in the same ballpark.

@PeterSommerlad
Copy link

Is there any chance we get const to the right?

@FrankHB
Copy link
Contributor

FrankHB commented Nov 3, 2014

What is the state of this issue? I found the inconsistency still exists widely even in same subclause, e.g. mutex(const mutex&) = delete; vs. lock_guard(lock_guard const&) = delete;.

@tkoeppe
Copy link
Contributor

tkoeppe commented Dec 5, 2015

Send pull requests?

tkoeppe added a commit that referenced this issue Nov 26, 2016
@tkoeppe
[re, thread] Move 'const' qualifier to the right place.
840ad53 
Cf. Issue #127.
tkoeppe added a commit that referenced this issue Nov 27, 2016
@tkoeppe
[re, thread] Move 'const' qualifier to the right place.
e864521 
Cf. Issue #127.
tkoeppe added a commit that referenced this issue Nov 27, 2016
@tkoeppe
[utilities] Harmonize spacing and placement of qualifiers
d8aab1f 
Fixes #127.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
C++14 DIS nice-to-haves
Development

No branches or pull requests
4 participants
@sdutoit @PeterSommerlad @FrankHB @tkoeppe