Skip to content

refactor: split SDI serial watch helpers#115

Open
dso2ng wants to merge 1 commit into
ch32-rs:mainfrom
dso2ng:refactor/sdi-watch-helper
Open

refactor: split SDI serial watch helpers#115
dso2ng wants to merge 1 commit into
ch32-rs:mainfrom
dso2ng:refactor/sdi-watch-helper

Conversation

@dso2ng

@dso2ng dso2ng commented Jun 24, 2026

Copy link
Copy Markdown

Summary

  • Split the SDI serial monitor path into smaller helpers for port discovery, output sinking, and timestamped printing.
  • Keep the existing watch_serial() public behavior and default WCH-Link CDC auto-detection unchanged.
  • Add unit tests for WCH-Link CDC VID/PID matching and timestamped line formatting across chunks.

Motivation

This makes the SDI serial watch path easier to extend in later changes, such as explicit port/probe selection or alternate output formats, while keeping this PR limited to an internal refactor.

Test Plan

  • cargo test
  • cargo build

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.

1 participant