Skip to content

Backport 3.6: Fix variable set but not used warning#10657

Open
bjwtaylor wants to merge 1 commit intoMbed-TLS:mbedtls-3.6from
bjwtaylor:unused-variable-ret-3.6
Open

Backport 3.6: Fix variable set but not used warning#10657
bjwtaylor wants to merge 1 commit intoMbed-TLS:mbedtls-3.6from
bjwtaylor:unused-variable-ret-3.6

Conversation

@bjwtaylor
Copy link
Copy Markdown

@bjwtaylor bjwtaylor commented Mar 25, 2026

Description

Add ret return pattern to avoid variable ret was set but never used warning. contrubutes #10448

PR checklist

  • changelog not required because: No public changes
  • development PR provided #Fix variable set but not used warning #10656
  • TF-PSA-Crypto PR not required because: No changes
  • framework PR not required
  • 3.6 PR provided #HERE
  • tests not required because: No changes

@bjwtaylor bjwtaylor changed the title Add ret return pattern to avoid variable ret was set but never used w… Fix variable set but not used warning Mar 25, 2026
…arning

Signed-off-by: Ben Taylor <ben.taylor@linaro.org>
@bjwtaylor bjwtaylor force-pushed the unused-variable-ret-3.6 branch from 0f946a0 to 3f35bb2 Compare March 25, 2026 15:55
@bjwtaylor bjwtaylor marked this pull request as ready for review March 26, 2026 07:58
@bjwtaylor bjwtaylor added 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 priority-medium Medium priority - this can be reviewed as time permits size-xs Estimated task size: extra small (a few hours at most) labels Mar 26, 2026
@gilles-peskine-arm gilles-peskine-arm changed the title Fix variable set but not used warning Backport 3.6: Fix variable set but not used warning Apr 10, 2026
Copy link
Copy Markdown
Contributor

@gilles-peskine-arm gilles-peskine-arm left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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 priority-medium Medium priority - this can be reviewed as time permits size-xs Estimated task size: extra small (a few hours at most)

Projects

Status: In Development

Development

Successfully merging this pull request may close these issues.

2 participants