Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Use size_t for variables/parameters representing size #2375

Closed
2 tasks done
thnkslprpt opened this issue Jun 15, 2023 · 0 comments · Fixed by #2376
Closed
2 tasks done

Use size_t for variables/parameters representing size #2375

thnkslprpt opened this issue Jun 15, 2023 · 0 comments · Fixed by #2376

Comments

@thnkslprpt
Copy link
Contributor

Checklist

  • I reviewed the Contributing Guide.
  • I performed a cursory search to see if the bug report is relevant, not redundant, nor in conflict with other tickets.

Describe the bug
Some variables/parameters in cFE representing size could be updated to the more expressive size_t typedef.

Most cases are already uint32 and are assigned values from other size_t types or sent into functions expecting size_t parameters anyway.

Expected behavior
Update to size_t where appropriate.

Reporter Info
Avi Weiss @thnkslprpt

dzbaker added a commit that referenced this issue Jul 1, 2024
…-variables-and-parameters

Fix #2375, Use size_t for variables/parameters representing size
@dzbaker dzbaker closed this as completed in 99eb3c3 Jul 2, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants