Skip to content

WIP: Started working on a post for reviewing unsafe code #8

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

Open
wants to merge 11 commits into
base: main
Choose a base branch
from
Open

WIP: Started working on a post for reviewing unsafe code #8

Changes from all commits
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
53260bb
Started working on a post for reviewing unsafe code
Michael-F-Bryan Jan 18, 2020
1021690
Reviewed lib.rs
Michael-F-Bryan Jan 18, 2020
f101fe6
Started analysing error.rs
Michael-F-Bryan Jan 20, 2020
f8c9675
Mentioned #[repr(transparent)] and making a PR for MessageError<E>'s …
Michael-F-Bryan Jan 21, 2020
f99f34c
Finished reviewing Error construction and downcasting
Michael-F-Bryan Jan 24, 2020
15d40c6
Reviewed the destructor
Michael-F-Bryan Jan 24, 2020
74b2f33
Started deconstructing the vtable code
Michael-F-Bryan Jan 25, 2020
24fa655
Started talking about `&unerased._object as *const E as *mut ()` and …
Michael-F-Bryan Jan 25, 2020
a2a5ee2
Mentioned the anyhow bug ticket
Michael-F-Bryan Jan 25, 2020
ac47f37
Added another copy of Error::downcast_mut()'s source code
Michael-F-Bryan Jan 25, 2020
b5c10fa
thiserror is still cool
Michael-F-Bryan Jan 25, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Loading