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

[C] Add support for constexpr storage class specifier #4071

Merged
merged 1 commit into from
Oct 30, 2024

Conversation

braewoods
Copy link
Contributor

This was added in C23 but is more limited in scope than the C++ version. The C version can only be applied to variables to indicate their usability as a compile time constant.

deathaxe
deathaxe previously approved these changes Oct 29, 2024
@michaelblyons
Copy link
Collaborator

If you're a big C user, would you be interested in looking at #1831, please? A lot of work was sunk into that by ismell, and you and other C users might be interested in the improvements it brings.

@braewoods
Copy link
Contributor Author

If you're a big C user, would you be interested in looking at #1831, please? A lot of work was sunk into that by ismell, and you and other C users might be interested in the improvements it brings.

I can after I go through my list of changes for newer C11 / C23 features. That said, while I am an experienced C developer, I'm still relatively new to the syntax definitions of sublime text so I'm not sure what I can achieve.

FichteFoll
FichteFoll previously approved these changes Oct 30, 2024
Copy link
Collaborator

@FichteFoll FichteFoll left a comment

Choose a reason for hiding this comment

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

@FichteFoll FichteFoll dismissed stale reviews from deathaxe and themself via 5351c24 October 30, 2024 14:30
This was added in C23 but is more limited in scope than the C++
version. The C version can only be applied to variables to indicate
their usability as a compile time constant.
@braewoods
Copy link
Contributor Author

I revamped the original commit to resolve previous conflict with a simpler rebase.

@deathaxe deathaxe merged commit 791fe94 into sublimehq:master Oct 30, 2024
1 check passed
@braewoods braewoods deleted the c-constexpr branch October 30, 2024 21:29
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.

4 participants