Skip to content

rdatomic: Fix var name typo in rd_atomic32_set - #5282

Open
Richard Lavoie (darkrift) wants to merge 1 commit into
confluentinc:masterfrom
darkrift:rdatomics_struct_typo
Open

Richard Lavoie (darkrift) wants to merge 1 commit into
confluentinc:masterfrom
darkrift:rdatomics_struct_typo

Conversation

@darkrift

Copy link
Copy Markdown

There is a typo in the variable name used to retrieve the member val of

Copilot AI review requested due to automatic review settings December 17, 2025 18:12
@darkrift
Richard Lavoie (darkrift) requested a review from a team as a code owner December 17, 2025 18:12

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes a typo in the rd_atomic32_set function where the variable name was incorrectly specified as rd instead of ra when retrieving the current value.

Key Changes

  • Corrected variable name from rd->val to ra->val in the rd_atomic32_set function

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@confluent-cla-assistant

confluent-cla-assistant Bot commented Dec 17, 2025

Copy link
Copy Markdown

🎉 All Contributor License Agreements have been signed. Ready to merge.
✅ darkrift
Please push an empty commit if you would like to re-run the checks to verify CLA status for all contributors.

@AvocadosConstant

Copy link
Copy Markdown

Also ran into this, was going to enter in the same fix. Can confirm that this fix is required for that conditional branch.

airlock-confluentinc Bot pushed a commit that referenced this pull request Jun 4, 2026
Document the OAuthBearer OIDC/CURL CMake compilation fix (#5135) and the
rd_atomic{32,64}_set previous-value fix that restores ALL_BROKERS_DOWN under
CMake (#5282).
airlock-confluentinc Bot pushed a commit that referenced this pull request Jun 25, 2026
Document the OAuthBearer OIDC/CURL CMake compilation fix (#5135) and the
rd_atomic{32,64}_set previous-value fix that restores ALL_BROKERS_DOWN under
CMake (#5282).
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.

3 participants