Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove unsafe code in argument parsing once Rust makes safe argument parsing possible #3

Closed
mortie opened this issue Apr 18, 2023 · 1 comment

Comments

@mortie
Copy link
Owner

mortie commented Apr 18, 2023

We currently use an unsafe transmute to work around the fact that OsString is opaque, since the argument to -i and -o are file paths which may be invalid UTF-8. Once rust-lang/rust#95290 or something like it is merged, we want to remove those uses of transmute.

@mortie
Copy link
Owner Author

mortie commented Jan 7, 2024

Rust implementation discontinued.

@mortie mortie closed this as not planned Won't fix, can't repro, duplicate, stale Jan 7, 2024
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

No branches or pull requests

1 participant