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

Remove unnecessary assignments #8953

Open
wants to merge 1 commit into
base: development
Choose a base branch
from

Conversation

rojer
Copy link
Contributor

@rojer rojer commented Mar 19, 2024

Description

Imporves readability and calms down clang-tidy.

PR checklist

Please tick as appropriate and edit the reasons (e.g.: "backport: not needed because this is a new feature")

  • changelog not required
  • 3.6 backport ???
  • 2.28 backport not required
  • tests not required

Reported by `clang-tidy`.

Signed-off-by: Deomid rojer Ryabkov <rojer@rojer.me>
@gilles-peskine-arm gilles-peskine-arm added enhancement needs-work needs-review Every commit must be reviewed by at least two team members, needs-ci Needs to pass CI tests priority-medium Medium priority - this can be reviewed as time permits size-xs Estimated task size: extra small (a few hours at most) needs-reviewer This PR needs someone to pick it up for review labels Mar 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement needs-ci Needs to pass CI tests 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 needs-work 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