◐ Shell
clean mode source ↗

Implement ascii codec in Rust by coolreader18 · Pull Request #3118 · RustPython/RustPython

Skip to content

Navigation Menu

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Sign up

Appearance settings

Conversation

@coolreader18

I started on this at the beginning of June, so I figured I should finish it and get it in. A large part of the refactoring is just deduplicating decoding logic between utf8 and ascii, since in both the valid subset can just be push_str()'d, and the rest passed to the error handler. The only difference is finding the invalid bytes, if any, and figuring out what the error is.

@coolreader18

Also sorry @fanninpm, I know you have #3046; that can be merged first if it's ready and I can rebase and resolve conflicts from there.

@fanninpm

that can be merged first if it's ready and I can rebase and resolve conflicts from there.

Actually, I would like to make use of the error-handling code you have here. This can go first so I can rebase #3046 on top of your work.

youknowone

Co-authored-by: Jeong YunWon <youknowone@users.noreply.github.com>

youknowone