Skip to content

[3.0.0 prep] Lossy error messages - #404

Draft
cicku wants to merge 3 commits into
rust-pcap:mainfrom
cicku:lossy-error-messages
Draft

[3.0.0 prep] Lossy error messages#404
cicku wants to merge 3 commits into
rust-pcap:mainfrom
cicku:lossy-error-messages

Conversation

@cicku

@cicku cicku commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

Discovered when writing tests for #403, w/ long non-ASCII paths. At 376 chars (976 UTF-8 bytes), std::fs created and read the file happily, but the crate failed with:

MalformedError(Utf8Error { valid_up_to: 254, error_len: None })

The issue exists in 2.5.0, but due to edition bump, PR must be merged after #400, and a rebase will be done then.

@cicku
cicku force-pushed the lossy-error-messages branch from 236e4bb to e874e61 Compare August 24, 2026 04:33
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