-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
extern types #1861
Merged
Merged
extern types #1861
Changes from 1 commit
Commits
Show all changes
19 commits
Select commit
Hold shift + click to select a range
6085dc4
Add extern types RFC
canndrew b2d8042
One sentance per line so future diffs are readable. 2 and typo fixes too
Ericson2314 e6f7f93
Expand teaching section
Ericson2314 083adad
Shout-out to roadmap with it's focus on FFI in the motivation section
Ericson2314 2f77316
Add ```rust in a few more places to get syntax highlighting
Ericson2314 949c292
Mention `size_of_val`/`align_of_val` in detailed design
Ericson2314 9f6d098
OOps
Ericson2314 cf183e9
"Sized" is the trait, not "?Sized"
Ericson2314 434b472
Talk about `c_void`, and associated representation concerns
Ericson2314 b3d5ba3
Merge pull request #1 from Ericson2314/extern-types
canndrew bce8a7b
Grammatic changes
canndrew 156ed4b
Remove indentation and invalid syntax
canndrew caa048b
Fix typo
canndrew d76b608
Mention OpaqueData as an alternative
canndrew a590767
grammar/typos
canndrew 9efddd3
Update with feedback
canndrew 263b7c9
Clarify that extern type pointers are thin pointers
canndrew e884a0b
re-word last commit
canndrew 4b971bd
RFC 1861 is Extern types
aturon File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably better to just say it is not a fat pointer and not even bother bringing up raw pointers because raw pointers can be fat too!
*const [T]
is fat for example despite being a raw pointer.