Skip to content

Fix #8, Remove unnecessary characters from memory dump event#79

Open
dzbaker wants to merge 1 commit into
nasa:mainfrom
dzbaker:fix-8
Open

Fix #8, Remove unnecessary characters from memory dump event#79
dzbaker wants to merge 1 commit into
nasa:mainfrom
dzbaker:fix-8

Conversation

@dzbaker

@dzbaker dzbaker commented Jul 11, 2023

Copy link
Copy Markdown
Contributor

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

dzbaker commented Jul 11, 2023

Copy link
Copy Markdown
Contributor Author

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

@thnkslprpt

Copy link
Copy Markdown
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