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

Backport 2.x: Replace reserved identifier clashes with suitable replacements #4779

Conversation

yutotakano
Copy link
Contributor

This is a backport of #4750, and it fixes #4630 for the 2.x maintenance branch. Identical edits were made by cherry-picking the commits, and verifying with a grep that the pattern existed nowhere.

It is a bug fix (and has a corresponding entry in the ChangeLog) because reserved identifiers had undefined behavior that could depend on the compiler. We eliminate any uncertainty in its behavior by making variable names conform to the requirements set by C.

Signed-off-by: Yuto Takano <yuto.takano@arm.com>
Signed-off-by: Yuto Takano <yuto.takano@arm.com>
Signed-off-by: Yuto Takano <yuto.takano@arm.com>
@yutotakano yutotakano added bug needs-review Every commit must be reviewed by at least two team members, Arm Contribution needs-reviewer This PR needs someone to pick it up for review size-s Estimated task size: small (~2d) labels Jul 14, 2021
@yutotakano yutotakano self-assigned this Jul 14, 2021
@gilles-peskine-arm gilles-peskine-arm added single-reviewer This PR qualifies for having only one reviewer and removed needs-review Every commit must be reviewed by at least two team members, needs-reviewer This PR needs someone to pick it up for review labels Jul 16, 2021
Copy link
Contributor

@ronald-cron-arm ronald-cron-arm left a comment

Choose a reason for hiding this comment

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

Fair backport of #4750.

@gilles-peskine-arm gilles-peskine-arm merged commit d9feb34 into Mbed-TLS:development_2.x Jul 22, 2021
@yutotakano yutotakano linked an issue Aug 4, 2021 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug single-reviewer This PR qualifies for having only one reviewer size-s Estimated task size: small (~2d)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Reserved identifiers used as local variables
3 participants