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

Motions 2019 11 cwg 2: P1968R0 Core Language Working Group "tentatively ready" Issues #3485

Merged
merged 19 commits into from Nov 20, 2019

Conversation

burblebee
Copy link
Contributor

@burblebee burblebee commented Nov 18, 2019

Fixes #3395.
Fixes #2664.

Issues:

  • CWG2419:
    In [expr.add]/bullet 4.2, and, after applying CWG2419, in [numeric.ops.midpoint]/p5 and [expr.rel]/p4, we have the wording:
    "a pointer to a hypothetical array element n"
    This reads like n is the hypothetical array element, but n is the size of the array.
    Can we reword this?
  • CWG2433: FYI, wording changed after Davis Herring's change "[basic.def.odr] Clean up new bullets"

@jensmaurer
Copy link
Member

CWG2419: What about "hypothetical n-th array element", except that gets us into the 0-th war. Maybe.

@jensmaurer jensmaurer added this to the post-2019-11 milestone Nov 19, 2019
@burblebee
Copy link
Contributor Author

burblebee commented Nov 19, 2019

@jensmaurer

CWG2419: What about "hypothetical n-th array element", except that gets us into the 0-th war. Maybe.

It's an improvement, but would have to be the "(n+1)th" element, which gets messy. I went with the existing wording that we have for this in [basic.compound] which uses x[n] to describe the element.

@zygoloid
Copy link
Member

The newest wording here is that in [expr.add], which uses "array element i of x"; I've updated all the other places to use that same formulation (example: "hypothetical array element n of x").

@zygoloid zygoloid merged commit 2978956 into master Nov 20, 2019
@jensmaurer jensmaurer deleted the motions-2019-11-cwg-2 branch February 18, 2020 20:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants