-
Notifications
You must be signed in to change notification settings - Fork 62
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
Fix #155, Remove unnecessary type translations #156
Fix #155, Remove unnecessary type translations #156
Conversation
@JackNWhite - could you confirm if this solves your issue (#155)? |
My apologies, but it doesn't fully work. Now, if the type is |
It should! I'll fix that. Thanks for checking. |
7d37c0a
to
c8766bc
Compare
CCB:2021-02-17 APPROVED
|
This pull request introduces 1 alert when merging c8766bc into f0dfdbd - view on LGTM.com new alerts:
|
c8766bc
to
4dcee36
Compare
4dcee36
to
2333fb8
Compare
@JackNWhite OK, was able to spot check this update... hopefully it works for you? |
I spot-checked and it worked for me, so as far as I know this is good to go. |
Bumps to: - cFE v6.8.0-rc1+dev365 - osal v5.2.0-rc1+dev280 - cFS-GroundSystem v2.2.0-rc1+dev29 See: - nasa/osal#830 - nasa/cFE#1171 - nasa/cFS-GroundSystem#162 Includes: - nasa/cFE#1148 - nasa/cFE#1168 - nasa/cFE#1158 - nasa/cFE#1151 - nasa/cFE#1173 - nasa/osal#794 - nasa/osal#818 - nasa/osal#820 - nasa/osal#815 - nasa/osal#813 - nasa/osal#822 - nasa/osal#823 - nasa/cFS-GroundSystem#156
Describe the contribution
Fix #155 - removes unnecessary type translations from CHeaderParser.py (also removes broken "double")
Testing performed
Requesting test by @JackNWhite
Spot checked processing of int32's so at least the pattern is ok (and runs)
Expected behavior changes
uint64
will no longer get translated byCHeaderParser
into the unsupporteddouble
typeSystem(s) tested on
cFS Dev Server, Ubuntu 18.04
Additional context
None
Third party code
None
Contributor Info - All information REQUIRED for consideration of pull request
Jacob Hageman - NASA/GSFC