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

Add from_raw_parts style interfaces to RelPtr and ArchivedBox #319

Merged
merged 4 commits into from
Oct 17, 2022

Conversation

fu5ha
Copy link
Contributor

@fu5ha fu5ha commented Oct 12, 2022

Discussed the need for this on Discord. Draft for now so I can prototype with it in protoss and make sure it works as expected/is necessary.

Also snuck in a ignore /tmp directory in .gitignore, a moderately common pattern which I use is to have rust-analyzer compile into a separate target directory so that it doesn't conflict with regular cargo builds.

@fu5ha
Copy link
Contributor Author

fu5ha commented Oct 12, 2022

:ohno: nightly compiler ICE-ing

@fu5ha fu5ha marked this pull request as ready for review October 12, 2022 17:37
@fu5ha
Copy link
Contributor Author

fu5ha commented Oct 12, 2022

Marked ready for review since it seems to work for what I want it to in protoss, though hasn't been super thoroughly tested. Maybe worth a MIRI run through :)

Not sure why it would cause the compiler to ICE.... I guess I'll just try rerunning and see if it was a fluke lol.

@fu5ha
Copy link
Contributor Author

fu5ha commented Oct 12, 2022

Seems the ICE is the same on rkyv without this commit on latest ntightly. Filed rust-lang/rust#102978 to track it...

@fu5ha
Copy link
Contributor Author

fu5ha commented Oct 17, 2022

The relevant ICE was in theory fixed, so should be good to go now :)

@djkoloski djkoloski merged commit 9f70aa3 into rkyv:master Oct 17, 2022
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

Successfully merging this pull request may close these issues.

None yet

2 participants