Posix build - lots of cleanup, and fixes pendant files problem#1673
Merged
MitchBradley merged 12 commits intomainfrom Feb 5, 2026
Merged
Posix build - lots of cleanup, and fixes pendant files problem#1673MitchBradley merged 12 commits intomainfrom
MitchBradley merged 12 commits intomainfrom
Conversation
The Posix build can serve as a base for automated tests. The vast majority of the changes were extant problems caught by a pickier compiler, including * Use snprintf instead of sprintf for safety * Missing or spurious "override" on virtual methods * A couple of buffer safety issues * Nonportable use of variable length arrays * Case-correct spelling of freertos Task.H and Queue.h * Type mismatches in printf arguments I had to change the way that undefinedPin and errorPin were defined to ensure that they are initialized prior to being used, which showed up with a different compiler. The changes that were directly related to Posix were quite small. A WinConsole.cpp and ExceptionHelper.cpp were moved from capture/ to (new) windows-x86/, and posix/Console.cpp was created to handle keyboard input.
Including a VFD simulator for the Huanyang protocol
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.