-
Notifications
You must be signed in to change notification settings - Fork 539
[WIP] Expressions rewrite #13
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
Merged
steveklabnik
merged 45 commits into
rust-lang:master
from
matthewjasper:expressions-rewrite
Jun 1, 2017
+836
−398
Merged
Changes from 1 commit
Commits
Show all changes
45 commits
Select commit
Hold shift + click to select a range
7e33fb9
Alway wrap text in expressions.md at 80 characters
matthewjasper 2ca41c6
Clean up sections on methods and functions
matthewjasper c993ff9
Make 'identifier's into links.
matthewjasper e924901
Some more corrections to lambda expressions
matthewjasper f8b3af3
Expand and correct the array expressions section.
matthewjasper c2e6797
Explain method call resolution
matthewjasper 7105048
Improve index expressions
matthewjasper 29af6c4
Improve unary operators
matthewjasper 4853a00
More wrapping changes
matthewjasper 837b41f
Merge operators
matthewjasper bcbba66
Improve binary operators
matthewjasper 5e4a958
Remove an unnecessary new blank line in an example
matthewjasper e091c57
Improve Compound assignments
matthewjasper a6048c3
Improve operator precedence.
matthewjasper 764da0e
Remove some unnecessary blank lines
matthewjasper a57aeea
Improve the example for grouped expressions
matthewjasper 056ffa6
Simlify and clarify method call resolution
matthewjasper f2ded24
Use 'then' instead of 'and' for ordered events.
matthewjasper e4102a3
Improve loops and control flow
matthewjasper d72c188
Move field expressions next to Methods
matthewjasper ed91f13
Make operator tables
matthewjasper 0e119ee
Add another example, fix a typo.
matthewjasper 6403330
Fix `while let` example.
matthewjasper ffecb50
More rvalue and lvalue stuff
36789c0
Get most of the potential content in place
227ac3f
More examples for method receivers.
3dda6f8
Add some links, fix typos
ca6fb9b
Fields can be borrowed/moved separately
e9fc5cd
More small improvements
80619cc
Explain that traits are duplicated in core once
ac2428e
Clarify that blocks are rvalues
b62a14e
Improve comparison examples
911b670
Yet more changes
matthewjasper 7c16ff2
Document constexpr and casts
matthewjasper e654bf5
Fix spelling, document rfcs 736, 968, 1229, 1535.
matthewjasper 8418895
FIx a mistake in what casts are allowed
matthewjasper e030480
Clean up some notes
matthewjasper 3c46293
Merge with more recent updates
matthewjasper 156a7f8
Fix links to loops secction
matthewjasper 5ea3d30
Remove most remaining lambda references
matthewjasper baf6f8e
Clean up some of the casts section
matthewjasper 7cc613d
Fix whitespace issues
matthewjasper 02ee9a6
Fix some things in the operator section
matthewjasper b75f52a
Fix tests
matthewjasper 4f39ffb
Make some small improvements to phrasing and examples
matthewjasper 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
Add another example, fix a typo.
- Loading branch information
commit 0e119ee7fba2923f90b74c023df5ad4aa7c6f74e
There are no files selected for viewing
This file contains hidden or 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.
Should I copy the (old) book explanation for all of the
as
casts here, or somewhere else?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.
here seems good