We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Is your feature request related to a problem? Please describe.
cfe/modules/core_api/fsw/inc/cfe_endian.h:66:12: error: result of '13398 << 24' requires 39 bits to represent, but 'int' only has 32 bits [-Werror=shift-overflow=] ((((n) << 24) & 0xFF000000) | (((n) << 8) & 0x00FF0000) | (((n) >> 8) & 0x0000FF00) | (((n) >> 24) & 0x000000FF))
From:
cFE/modules/core_api/fsw/inc/cfe_endian.h
Lines 64 to 66 in 10898c9
Describe the solution you'd like Mask, then shift.
Describe alternatives you've considered None
Additional context
Or really transition to serializer/deserializaer concepts.
Requester Info Jacob Hageman - NASA/GSFC
The text was updated successfully, but these errors were encountered:
Fix nasa#2118, Endian macro mask before shift to avoid shift overflow…
8633f43
… warning
41897e1
0286b6e
Merge pull request #2119 from skliper/fix2118-endian_mask
68931c6
Fix #2118, Endian macro mask before shift to avoid shift overflow warning
skliper
Successfully merging a pull request may close this issue.
Is your feature request related to a problem? Please describe.
From:
cFE/modules/core_api/fsw/inc/cfe_endian.h
Lines 64 to 66 in 10898c9
Describe the solution you'd like
Mask, then shift.
Describe alternatives you've considered
None
Additional context
Or really transition to serializer/deserializaer concepts.
Requester Info
Jacob Hageman - NASA/GSFC
The text was updated successfully, but these errors were encountered: