Skip to content

implement v3 - #26

Merged
Katze719 merged 11 commits into
mainfrom
feat/implement-v3
Sep 10, 2026
Merged

implement v3#26
Katze719 merged 11 commits into
mainfrom
feat/implement-v3

Conversation

@Katze719

@Katze719 Katze719 commented Sep 6, 2026

Copy link
Copy Markdown
Member
  • Updated serialSetFlowControl, serialSetParity, and serialSetStopBits functions to accept typed enums instead of integers for better type safety.
  • Introduced serialWaitForDrain function to ensure all output data is transmitted before proceeding.
  • Modified serialWrite function to accept a SerialTimeoutConfig structure instead of separate timeout parameters.
  • Updated unit tests and integration tests to reflect changes in function signatures and ensure proper handling of timeout configurations.
  • Added new test cases for serialWaitForDrain and refactored existing tests to use the new configurations.
  • Ensured all relevant files include necessary headers for the new implementations.

- Updated `serialSetFlowControl`, `serialSetParity`, and `serialSetStopBits` functions to accept typed enums instead of integers for better type safety.
- Introduced `serialWaitForDrain` function to ensure all output data is transmitted before proceeding.
- Modified `serialWrite` function to accept a `SerialTimeoutConfig` structure instead of separate timeout parameters.
- Updated unit tests and integration tests to reflect changes in function signatures and ensure proper handling of timeout configurations.
- Added new test cases for `serialWaitForDrain` and refactored existing tests to use the new configurations.
- Ensured all relevant files include necessary headers for the new implementations.
@Katze719 Katze719 self-assigned this Sep 6, 2026
@Katze719 Katze719 added enhancement New feature or improvement breaking change Change requires users to adapt area: native api OS-specific implementation and exported native API area: ffi metadata ASTrein metadata generation and FFI surface area: build & ci CMake, toolchains, workflows, and release automation labels Sep 6, 2026
@Katze719
Katze719 requested a review from Mqxx September 7, 2026 18:39
@Katze719 Katze719 changed the title Refactor serial communication functions to use typed configurations implement v3 Sep 7, 2026
Comment thread integration_tests/integration_test.ts Outdated
Comment thread integration_tests/ffi_bindings.ts
Comment thread integration_tests/ffi_bindings.ts Outdated
Comment thread src/serial_set_event_callback.cpp Outdated
@Katze719
Katze719 requested a review from Mqxx September 8, 2026 20:09
Comment thread jsr/README.md Outdated
Comment thread src/serial_v3_api.test.cpp Outdated
@Katze719
Katze719 requested a review from Mqxx September 9, 2026 06:44

@Mqxx Mqxx left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

Comment thread src/serial_extended_api.test.cpp
@Katze719
Katze719 merged commit 7beb0eb into main Sep 10, 2026
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: build & ci CMake, toolchains, workflows, and release automation area: ffi metadata ASTrein metadata generation and FFI surface area: native api OS-specific implementation and exported native API breaking change Change requires users to adapt enhancement New feature or improvement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants