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

[dcl.link] p10 objects vs. entities #7386

Merged
merged 1 commit into from
Nov 20, 2024
Merged

[dcl.link] p10 objects vs. entities #7386

merged 1 commit into from
Nov 20, 2024
+2 −2

Conversation

xmh0511
Copy link
Contributor

@xmh0511 xmh0511 commented Nov 14, 2024 •
edited
Loading

[dcl.link] p10 says:

Linkage from C++ to objects defined in other languages and to objects defined in C++ from other languages is implementation-defined and language-dependent.
Only where the object layout strategies of two language implementations are similar enough can such linkage be achieved.

The objects should be changed to entities, which includes both objects and functions for which the linkage is applied.

Linkage from C++ to entities defined in other languages and to entities defined in C++ from other languages is implementation-defined and language-dependent.

@xmh0511
objects vs. entities

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
GPG key ID: B5690EEEBB952194
Verified
Learn about vigilant mode
Loading
Loading status checks…
5b5d297
@xmh0511 xmh0511 mentioned this pull request Nov 14, 2024
Open
@jwakely
Copy link
Member

jwakely commented Nov 14, 2024

Please use the usual style as requested in https://github.com/cplusplus/draft/wiki/How-to-submit-an-editorial-issue

@xmh0511 xmh0511 changed the title objects vs. entities [dcl.link] p10 objects vs. entities Nov 14, 2024
@xmh0511
Copy link
Contributor Author

xmh0511 commented Nov 14, 2024

Please use the usual style as requested in https://github.com/cplusplus/draft/wiki/How-to-submit-an-editorial-issue

updated.

@tkoeppe
Copy link
Contributor

tkoeppe commented Nov 20, 2024

It is the commit message that we'd like to see updated, not the PR title. (The PR title is not terribly important.)

@tkoeppe tkoeppe merged commit f4c4c7c into cplusplus:main Nov 20, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jensmaurer jensmaurer

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@xmh0511 @jwakely @tkoeppe @jensmaurer