-
Notifications
You must be signed in to change notification settings - Fork 840
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
error: casting &T
to &mut T
is undefined behavior
#1485
Comments
While the code in this repo should be fixed, a temporary workaround is to use an older version of the rust toolchain (I had success with rust 1.72.0, installing version 0.13.2):
Originally I was trying to install 0.13.3, but ran into issues because the |
This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days. |
making github-actions close an obvious bug just because no one made comments on it doesn't make the bug go away. |
Pretty sure this was fixed |
rust 1:1.78.0-1
|
oh i see this fix in lastest code |
For future reference, I think this was fixed in commit 4322056 |
Since this came up in the Rust community discord: The code is still unsound. let word = &words[i] as *const _ as *mut Word; Writing to this pointer is immediate undefined behavior. You may not modify the value behind a I would recommend to instead collect all the indices into a HashSet, and then use
|
PR will be merged soon and will release! |
ERROR: Failed building wheel for tokenizers:
Full log: https://snips.sh/f/XPSNFeHd9Q
Python 3.12.2
rustc 1.76.0 (07dca489a 2024-02-04) (Void Linux)
The text was updated successfully, but these errors were encountered: