Skip to content

Implement serial communication functions and refactor existing code - #19

Merged
Katze719 merged 14 commits into
mainfrom
upgrade/cpp-core-v2-linux
Aug 23, 2026
Merged

Implement serial communication functions and refactor existing code#19
Katze719 merged 14 commits into
mainfrom
upgrade/cpp-core-v2-linux

Conversation

@Katze719

Copy link
Copy Markdown
Member
  • Added new functions for serial communication including:

    • serialOutBytesTotal
    • serialOutBytesWaiting
    • serialReadLine
    • serialReadUntil
    • serialReadUntilSequence
    • serialSendBreak
    • serialSetBaudrate
    • serialSetDataBits
    • serialSetDtr
    • serialSetFlowControl
    • serialSetParity
    • serialSetReadCallback
    • serialSetRts
    • serialSetStopBits
    • serialSetWriteCallback
  • Refactored serialRead and serialWrite functions to use a common implementation (readImpl and writeImpl).

  • Updated error handling to use new status code definitions.

  • Modified unit tests to reflect changes in error codes and added new tests for the newly implemented functions.

  • Ensured consistent naming and usage of status codes across the codebase.

Katze719 and others added 8 commits June 17, 2026 22:36
- Added new functions for serial communication including:
  - `serialOutBytesTotal`
  - `serialOutBytesWaiting`
  - `serialReadLine`
  - `serialReadUntil`
  - `serialReadUntilSequence`
  - `serialSendBreak`
  - `serialSetBaudrate`
  - `serialSetDataBits`
  - `serialSetDtr`
  - `serialSetFlowControl`
  - `serialSetParity`
  - `serialSetReadCallback`
  - `serialSetRts`
  - `serialSetStopBits`
  - `serialSetWriteCallback`

- Refactored `serialRead` and `serialWrite` functions to use a common implementation (`readImpl` and `writeImpl`).
- Updated error handling to use new status code definitions.
- Modified unit tests to reflect changes in error codes and added new tests for the newly implemented functions.
- Ensured consistent naming and usage of status codes across the codebase.
@Katze719
Katze719 marked this pull request as ready for review June 22, 2026 16:36
@Katze719
Katze719 requested a review from Mqxx June 22, 2026 16:36

@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.

Not done yet with all files

Comment thread .github/workflows/build_binary.yml
Comment thread .github/workflows/test_deno.yml
Comment thread .github/workflows/test_deno.yml
Comment thread src/detail/posix_helpers.hpp Outdated
Comment thread src/detail/posix_helpers.hpp Outdated
Comment thread src/serial_abort_read.cpp Outdated
Comment thread src/serial_abort_write.cpp Outdated
Comment thread src/serial_clear_buffer_in.cpp Outdated
Comment thread src/serial_set_dtr.cpp Outdated
Comment thread src/serial_set_flow_control.cpp Outdated
Comment thread src/serial_set_parity.cpp Outdated
Comment thread src/serial_set_rts.cpp Outdated
Comment thread src/serial_set_stop_bits.cpp Outdated
@Katze719
Katze719 requested a review from Mqxx June 27, 2026 17:32
Comment thread src/detail/posix_common.hpp Outdated
@Katze719
Katze719 requested a review from Mqxx July 15, 2026 09:38
Comment thread src/detail/posix_parse_stop_bits.hpp Outdated
Comment thread src/serial_monitor_ports.cpp
Comment thread src/serial_set_error_callback.cpp
Comment thread .github/workflows/test_deno.yml Outdated
Comment thread src/detail/acquire_handle_context.hpp
@Katze719
Katze719 requested a review from Mqxx August 23, 2026 18:50
@Katze719
Katze719 merged commit 8ca3cf4 into main Aug 23, 2026
7 checks passed
@Katze719
Katze719 deleted the upgrade/cpp-core-v2-linux branch August 23, 2026 19:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants