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

LinuxGpioDriver: fix static_assert's for FwSizeType. Possible fix for #3030 #3118

Open
wants to merge 2 commits into
base: devel
Choose a base branch
from

Conversation

AlesKus
Copy link
Contributor

@AlesKus AlesKus commented Jan 8, 2025

Related Issue(s) #3030
Has Unit Tests (y/n) n
Documentation Included (y/n) n

Change Description

A possible fix to avoid warning for signed/unsigned comparison in LinuxGpioDriver.
The signed max() values are cast to corresponding unsigned types. The cast is safe because max() returns nonnegative value which shall fit to unsigned type.

Rationale

Possible fix for issue #3030.

Testing/Review Recommendations

Steps to reproduce and test:

  1. cd to the top level directory of the fprime repo
  2. fprime-util generate -f
  3. fprime-util build

There should be no build errors.

Future Work

N/A.

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.

1 participant