Uh oh!
There was an error while loading. Please reload this page.
Bring Windows bindings to Linux feature parity - #10
Merged
Conversation
…nd control - Added `serialGetConfig` to retrieve serial port configuration. - Implemented `serialGetCts`, `serialGetDcd`, `serialGetDsr`, and `serialGetRi` to check modem status. - Introduced `serialMonitorPorts` for monitoring available COM ports. - Created functions for controlling DTR, RTS, and flow control settings. - Added `serialSendBreak` and `serialUpdateBaudrate` for sending break signals and updating baud rates.
Katze719
marked this pull request as draft
August 26, 2026 20:30
Katze719
marked this pull request as ready for review
August 26, 2026 20:31
…erial communication - Added functions for registering and removing handle states. - Implemented abort request handling for pending operations. - Created utilities for starting and waiting on pending I/O operations. - Added conversion functions between UTF-8 and wide strings. - Developed a validation function for Win32 handles. - Introduced error handling utilities for Win32 API calls. - Refactored existing serial communication functions to utilize new handle management and I/O utilities. - Removed deprecated win32_helpers.hpp and consolidated functionality into new headers. - Updated serial communication interface to improve error handling and state management.
Katze719
marked this pull request as draft
August 28, 2026 21:58
Katze719force-pushed
the
feat/windows-linux-parity
branch
from
August 28, 2026 22:53
85eb505 to
7fd7638Compare…ming and improve error handling - Updated variable names from 'h' to 'native_handle' for clarity across multiple serial functions. - Changed return values to use 'status' for error handling consistency. - Refactored DCB structure usage to 'serial_settings' for better readability. - Enhanced readability in the serial port listing and monitoring functions by renaming variables and improving callback function names. - Improved test code for better clarity and consistency in variable naming.
…t in build configuration
… in build configuration
Katze719
marked this pull request as ready for review
August 31, 2026 20:27
Mqxx
reviewed
Sep 1, 2026
Uh oh!
There was an error while loading. Please reload this page.
Mqxx
approved these changes
Sep 1, 2026
Uh oh!
There was an error while loading. Please reload this page.
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 freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Summary