Skip to content

Conversation

@PastaPastaPasta
Copy link
Member

Modified the definition of MI_ATOMIC_VAR_INIT to handle cases where ATOMIC_VAR_INIT is not defined or when compiling with C23, where it has been removed. This change ensures compatibility with newer C standards while maintaining support for older versions.

Issue being fixed or feature implemented

  • Why is this change required? What problem does it solve?
  • If it fixes an open issue, please link to the issue here.

What was done?

Describe your changes in detail

How Has This Been Tested?

Please describe in detail how you tested your changes.

Include details of your testing environment, and the tests you ran
to see how your change affects other areas of the code, etc.

Breaking Changes

Please describe any breaking changes your code introduces

Checklist:

Go over all the following points, and put an x in all the boxes that apply.

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have added or updated relevant unit/integration/functional/e2e tests
  • I have made corresponding changes to the documentation
  • I have assigned this pull request to a milestone (for repository code-owners and collaborators only)

Modified the definition of MI_ATOMIC_VAR_INIT to handle cases where ATOMIC_VAR_INIT is not defined or when compiling with C23, where it has been removed. This change ensures compatibility with newer C standards while maintaining support for older versions.
@github-actions
Copy link

github-actions bot commented Nov 4, 2025

✅ No Merge Conflicts Detected

This PR currently has no conflicts with other open PRs.

@PastaPastaPasta
Copy link
Member Author

@knst can you investigate if there's an issue with bleeding edge compilers?

Copy link
Collaborator

@kwvg kwvg left a comment

Choose a reason for hiding this comment

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

See bls-signatures#116 for alternative

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