Skip to content

Fix BnStrCompatible impl for &Path #6369

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.

Sign up for GitHub

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

Jump to bottom
Merged
emesare merged 1 commit into from
Jan 28, 2025
Merged

Fix BnStrCompatible impl for &Path #6369

emesare merged 1 commit into from
Jan 28, 2025

Conversation

@mkrasnitski
Copy link
Contributor

@mkrasnitski mkrasnitski commented Jan 28, 2025

Turns out OsStr isn't null terminated...

@emesare emesare self-assigned this Jan 28, 2025
@emesare emesare added the Component: Rust API Issue needs changes to the Rust API label Jan 28, 2025
@emesare emesare merged commit 7fc1623 into Vector35:dev Jan 28, 2025
4 of 5 checks passed
@mkrasnitski mkrasnitski deleted the path-quickfix branch January 28, 2025 18:10
@emesare emesare added this to the Gallifrey milestone Apr 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@emesare emesare

Labels

Component: Rust API Issue needs changes to the Rust API

Projects

None yet

Milestone

Gallifrey

Development

Successfully merging this pull request may close these issues.

2 participants

@mkrasnitski @emesare