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

[rand.adapt.shuf] Declaration and initialization order of "exposition only" variables does not match #1331

Closed
Eelis opened this issue Jan 4, 2017 · 0 comments
Closed

[rand.adapt.shuf] Declaration and initialization order of "exposition only" variables does not match #1331

Eelis opened this issue Jan 4, 2017 · 0 comments
Assignees
@jensmaurer

Comments

@Eelis
Copy link
Contributor

Eelis commented Jan 4, 2017

[rand.adapt.shuf]#6 states that the constructors initialize Y after V, but #3 declares Y before V.

The variables are "exposition only", but it's still a bit sloppy. :)
@jensmaurer jensmaurer self-assigned this Jan 4, 2017
jensmaurer added a commit to jensmaurer/draft that referenced this issue Jan 4, 2017
@jensmaurer
[rand.adapt.shuf] Show exposition-only members in the order
d256653 
in which initialization is described.

Fixes cplusplus#1331.
@jensmaurer jensmaurer mentioned this issue Jan 4, 2017
Merged
tkoeppe pushed a commit that referenced this issue Jan 5, 2017
@jensmaurer @tkoeppe
[rand.adapt.shuf] Show exposition-only members in the order in which …
732fd44 
…initialization is described. (#1332)

Fixes #1331.
burblebee pushed a commit that referenced this issue Nov 11, 2017
P0641R2 Resolving Core Issue #1331 (const mismatch with defaulted cop…
6308467 
…y constructor)

Also fixes CWG1331 and CWG1426.

[dcl.fct.def.default] Fix English by adding "had".
zygoloid pushed a commit that referenced this issue Nov 17, 2017
@zygoloid
P0641R2 Resolving Core Issue #1331 (const mismatch with defaulted cop…
eb5fe3a 
…y constructor)

Also fixes CWG1331 and CWG1426.

[dcl.fct.def.default] Fix English by adding "had".
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jensmaurer jensmaurer

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Eelis @jensmaurer