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

Fix #22, Use %p for printing addresses, resolves truncation #26

Merged
merged 1 commit into from
Jun 1, 2022

Conversation

skliper
Copy link
Contributor

@skliper skliper commented May 27, 2022

Checklist (Please check before submitting)

Describe the contribution

Testing performed
Built and ran unit tests, passed

Expected behavior changes
No longer truncates addresses in event messages

System(s) tested on

  • Hardware: i5/wsl
  • OS: Ubuntu 18.04
  • Versions: Bundle main + this commit (w/ app additions)

Additional context
None

Third party code
None

Contributor Info - All information REQUIRED for consideration of pull request
Jacob Hageman - NASA/GSFC

@skliper skliper added this to the Draco milestone May 27, 2022
@astrogeco
Copy link
Contributor

CCB:2022-06-01 APPROVED

@astrogeco astrogeco merged commit 3b474d3 into nasa:main Jun 1, 2022
@skliper skliper deleted the fix22-address_truncation branch June 10, 2022 13:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

MM Messages - Memory address being truncated to 32 bits on 64 bit machines
2 participants