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

[dcl.array] Clarify that arrays do not have extra padding. #4061

Merged
merged 1 commit into from Oct 6, 2020

Conversation

jensmaurer
Copy link
Member

Fixes #3791

@jensmaurer jensmaurer added the tiny An issue with a small change; with "cwg" label: can be applied editorially after CWG consent. label Jun 26, 2020
@@ -3084,7 +3084,7 @@
\end{note}

\pnum
An object of type ``array of \tcode{N} \tcode{U}'' contains
An object of type ``array of \tcode{N} \tcode{U}'' consists of
a contiguously allocated non-empty set
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is «contiguously allocated» normative? If not, could it be fixed editorially in this PR or it should be a separate PR?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What's wrong with "contiguously allocated"?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've thought we agreed in core reflector that it is not normative, like the wording about how member subobjects are allocated.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't remember. Do you have a pointer?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The participation on the core reflector was low for this question. I'd like to restrict this pull request to the obvious fix "contains" -> "consists of".

@zygoloid zygoloid added this to the C++23 milestone Sep 18, 2020
@zygoloid zygoloid merged commit 1536e33 into cplusplus:master Oct 6, 2020
@jensmaurer jensmaurer deleted the c30 branch October 6, 2020 20:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tiny An issue with a small change; with "cwg" label: can be applied editorially after CWG consent.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[dcl.array] Potentially ambiguous specification of array contents
3 participants