Skip to content

C2Y: member access clarification#147

Merged
cdervis merged 2 commits intocppstat:mainfrom
uecker:uecker-patch-8
Mar 31, 2026
Merged

C2Y: member access clarification#147
cdervis merged 2 commits intocppstat:mainfrom
uecker:uecker-patch-8

Conversation

@uecker
Copy link
Copy Markdown
Collaborator

@uecker uecker commented Mar 31, 2026

Every compiler does this anyway
https://godbolt.org/z/o1zqcrddG

Every compiler does this anyway
https://godbolt.org/z/o1zqcrddG

Signed-off-by: Martin Uecker <[email protected]>
@cdervis
Copy link
Copy Markdown
Member

cdervis commented Mar 31, 2026

Could you please add MSVC to the list as well? And just remove the 16.4 version of Xcode, making this feature complete.

@uecker
Copy link
Copy Markdown
Collaborator Author

uecker commented Mar 31, 2026

I wonder whether we should just remove entries that are not really feature but essentially wording cleanups?

@cdervis
Copy link
Copy Markdown
Member

cdervis commented Mar 31, 2026

I'm for it, especially if they're behaviors that were always implemented by all major toolchains anyway. I recently removed various C++ papers that were just wordings/clarifications, since they would just clutter the tables.

We can do the same for C, although C has fewer features in general and serves a different niche than C++, so they could be tolerable.

I think we can leave the features in for now, and then remove all the irrelevant ones at once during a larger review session later.

Edit: For protocol, I have created Issue #150.

@cdervis cdervis merged commit 3720afd into cppstat:main Mar 31, 2026
1 check passed
@uecker
Copy link
Copy Markdown
Collaborator Author

uecker commented Mar 31, 2026

So you want those fixes or not?

@cdervis
Copy link
Copy Markdown
Member

cdervis commented Mar 31, 2026

Which fixes? This PR has been merged.

@uecker
Copy link
Copy Markdown
Collaborator Author

uecker commented Mar 31, 2026

It seems you have reset the branch here, or did I miss something: https://github.com/cppstat/cppstat

@cdervis
Copy link
Copy Markdown
Member

cdervis commented Mar 31, 2026

Thanks for letting me know, seems like the merges got mixed up with my local, unpublished changes. I've pushed your contributions up again.

@uecker
Copy link
Copy Markdown
Collaborator Author

uecker commented Mar 31, 2026

Ah, good. I thought I was pushing something you didn't like. Should I create pull requests or push some changes directly?

@cdervis
Copy link
Copy Markdown
Member

cdervis commented Mar 31, 2026

All good, it was just my bad. Since you're an author on various proposals, I've given you write access. PRs are preferred, since the CI may catch typos and invalid input in general. You can merge your own PRs as you wish.

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

Successfully merging this pull request may close these issues.

2 participants