BurntSushi / BurntSushi/encoding_rs_io

implement Read::read_to_string on top of Decoder::decode_to_string

Open
#4 0 comments 0 reactions 0 assignees View on GitHub
enhancement good first issue help wanted
Dominant language
Rust
Stars
30
Forks
10
PR merge metrics
No merged PRs in 30d

Description

This is a possibly somewhat common method, and the decoder in this crate could implement it more efficiently by avoiding UTF-8 re-validation. See @hsivonen's comment here: https://github.com/hsivonen/encoding_rs/issues/8#issuecomment-411356371

Contributor guide

No contributing guide indexed for this repository

Research direction

Locate the implementations of Read::read_to_string and Decoder::decode_to_string in this crate, then read the linked encoding_rs discussion for the intended efficiency constraint. Confirm how the adapter currently handles decoded text and use the existing test suite to verify that read_to_string works without unnecessary UTF-8 re-validation.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
backend
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.