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

[expr.ass] Rename problematic stable label #7524

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

AlisdairM
Copy link
Contributor

As more folks from outside the technically focused WG21 committee pay attention to our standard, some language starts to gain attention as problematic. This commit fixes one such case.

@AlisdairM AlisdairM force-pushed the rename_problematic_label branch from ebf9fe7 to 34083d2 Compare December 29, 2024 04:55
Copy link
Member

@jensmaurer jensmaurer left a comment

Choose a reason for hiding this comment

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

Some members of CWG have expressed (unsolicited) support for this rename while CWG was discussing [stmt] and [dcl].

@tkoeppe , does that suffice for you, or do you want a more "official" statement from CWG?

@shafik
Copy link

shafik commented Jan 10, 2025

Some members of CWG have expressed (unsolicited) support for this rename while CWG was discussing [stmt] and [dcl].

@tkoeppe , does that suffice for you, or do you want a more "official" statement from CWG?

Jens, I think I would like to have a wider discussion of the impact of this change to the wider C++ community.

This change has a wider blast radius considering how many Stackoverflow, blogs posts and other online community content we have out there that relies solely on the stable names to refer to standard. While folks more intimately connected to the WG21 process likely will catch on quickly enough. It is unlikely for most of the existing content out there to be updated quickly or ever at all.

As someone who came to WG21 slowly over time by participating in various C++ communities I see how opaque the WG21 process can be to those not involved. So this has the potential to make a lot of content a lot less useful for folks far away from the process.

Having spent also a lot of time in C community early on, I have seen how not having stable names has made a lot of old C content much less usable. So the impact is very real, they are stable names for a reason.

I don't think we should be willing to willy nilly go about updating stable names without some deeper thought.

Maybe we want to accept that cost but we should discuss that.

@jensmaurer jensmaurer added the cwg Issue must be reviewed by CWG. label Jan 11, 2025
@jensmaurer
Copy link
Member

Ack. Sending e-mail to EWG and CWG.

As more folks from outside the technically focused WG21 committee
pay attention to our standard, some language starts to gain attention
as problematic.  This commit fixes one such case.
@AlisdairM AlisdairM force-pushed the rename_problematic_label branch from 34083d2 to a05758b Compare January 15, 2025 19:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cwg Issue must be reviewed by CWG.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants