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

[2018-06 LWG Motion 28] P0898R3 Standard library concepts #2145

Closed
jensmaurer opened this issue Jun 9, 2018 · 12 comments
Closed

[2018-06 LWG Motion 28] P0898R3 Standard library concepts #2145

jensmaurer opened this issue Jun 9, 2018 · 12 comments
Assignees
Milestone

Comments

@jensmaurer
Copy link
Member

jensmaurer commented Jun 9, 2018

P0898R3

@jensmaurer jensmaurer added this to the post-2018-06 milestone Jun 9, 2018
@tkoeppe
Copy link
Contributor

tkoeppe commented Jun 9, 2018

We should get the LaTeX sources from @CaseyCarter. I'd very much like to have an authoritative source for the stable labels that are used throughout the new wording.

@jensmaurer
Copy link
Member Author

We still have to double-check the labels as per our current policy.

@CaseyCarter
Copy link
Contributor

I plan to cleanup my latex sources and prepare the PR for P0898. Could someone please assign this issue to me so there's no confusion among the editors?

@jensmaurer
Copy link
Member Author

Apparently, you can't assign to foreign people :-(

Assigning to myself to mark it as "taken" in the interim.

@jensmaurer jensmaurer self-assigned this Jun 9, 2018
@zygoloid zygoloid assigned CaseyCarter and unassigned jensmaurer Jun 9, 2018
@zygoloid
Copy link
Member

zygoloid commented Jun 9, 2018

I've reconfigured GitHub so all editors can be assigned issues on this repository.

@CaseyCarter
Copy link
Contributor

@timsong-cpp has pointed out to me that [meta.trans.other]/3.2:

[let] XREF(A) denote a unary class template T such that T<U> denotes the same type as U with the addition of A’s cv and reference qualifiers, for a type U such that is_same_v<U, remove_cvref_t<U>> is true

isn't possibly implementable with a class template, and that it should rather read "unary alias template". I'll file an issue to make this change after the merge.

@tkoeppe
Copy link
Contributor

tkoeppe commented Jun 11, 2018

@CaseyCarter: is that editorial? If so, we could just tack that onto the motion application. Otherwise we'll wait for the LWG issue.

@CaseyCarter
Copy link
Contributor

is that editorial?

By the strictest possible interpretation it is not editorial. It is certainly a borderline case: I would have made the change in the proposal had it been brought to my attention before the 20:00 deadline for the strawpolls page. I don't know where we draw the line for changes we consider to be "editorial-ish-enough" when incorporating motions, so I plan to precisely walk the line absent any direction otherwise from more experienced editors.

@tkoeppe
Copy link
Contributor

tkoeppe commented Jun 11, 2018

@mclow: Maybe the LWG chair could weigh in on this border line?

@mclow
Copy link
Contributor

mclow commented Jun 11, 2018

I think we should err on the side of caution here - and open an issue.

@CaseyCarter
Copy link
Contributor

@timsong-cpp again:

Hmm, where in CopyConstructible do we say that the construction must not change v?

This axiom used to be implicit via the "const operands are not modified by default" wording when CopyConstructible was defined in terms of required expressions; it was lost when we reformulated in terms of is_constructible. I need to open an issue to fix this as well.

@CaseyCarter
Copy link
Contributor

I think we should err on the side of caution here - and open an issue.

This is LWG 3140.

Hmm, where in CopyConstructible do we say that the construction must not change v?

This is LWG 3141

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

No branches or pull requests

5 participants