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 #8, Remove unnecessary characters from memory dump event #79

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

dzbaker
Copy link
Contributor

@dzbaker dzbaker commented Jul 11, 2023

Checklist

Describe the contribution

Testing performed
Build & Run + Unit Tests.

Expected behavior changes
Increases the number of bytes that can be dumped in the event message string dramatically (by around double, depending on the value that is set for CFE_MISSION_EVS_MAX_MESSAGE_LENGTH).

Contributor Info
@thnkslprpt

@dzbaker
Copy link
Contributor Author

dzbaker commented Jul 11, 2023

@thnkslprpt I reverted the original (#52) so we could move it to the next release. Does everything look good here?

@thnkslprpt
Copy link
Contributor

@thnkslprpt I reverted the original (#52) so we could move it to the next release. Does everything look good here?

Yes looks great Dylan.
Cheers

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_DumpInEventCmd wastes event characters
2 participants