Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
tutorial: clearer explanation of freezing.
- "Lending an immutable pointer" might be confusing. It was not discussed why borrowed pointers are immutable in the first place. - Make it clear that the borrowed pointers are immutable even if the variable was declared with `mut`. - Make it clear that we cannot even assign anything to the variable while its value is being borrowed. tutorial: change "--" to an em-dash. tutorial: change instances of "--" to em-dash.
- Loading branch information
82747ed
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.
saw approval from pnkfelix
at https://github.com/jagtalon/rust/commit/82747ed93e5af967b691653a56849da2c209d85d
82747ed
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.
merging jagtalon/rust/jag/rust/tutorial-freezing = 82747ed into auto
82747ed
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.
jagtalon/rust/jag/rust/tutorial-freezing = 82747ed merged ok, testing candidate = 3216df86
82747ed
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.
some tests failed:
failure: http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/4504
exception: http://buildbot.rust-lang.org/builders/auto-mac-32-opt/builds/4489
exception: http://buildbot.rust-lang.org/builders/auto-mac-32-nopt-t/builds/2332
exception: http://buildbot.rust-lang.org/builders/auto-mac-64-opt/builds/4487
exception: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-c/builds/3587
exception: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-t/builds/3593
exception: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/4502
exception: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/3596
exception: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-t/builds/3595
exception: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android-t/builds/1443
exception: http://buildbot.rust-lang.org/builders/auto-win-32-opt/builds/4496
exception: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/3603
exception: http://buildbot.rust-lang.org/builders/auto-bsd-64-opt/builds/4263
82747ed
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.
saw approval from pnkfelix
at https://github.com/jagtalon/rust/commit/82747ed93e5af967b691653a56849da2c209d85d
82747ed
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.
merging jagtalon/rust/jag/rust/tutorial-freezing = 82747ed into auto
82747ed
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.
jagtalon/rust/jag/rust/tutorial-freezing = 82747ed merged ok, testing candidate = 6c41f99
82747ed
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.
all tests pass:
success: http://buildbot.rust-lang.org/builders/auto-mac-32-opt/builds/4490
success: http://buildbot.rust-lang.org/builders/auto-mac-32-nopt-c/builds/2332
success: http://buildbot.rust-lang.org/builders/auto-mac-32-nopt-t/builds/2333
success: http://buildbot.rust-lang.org/builders/auto-mac-64-opt/builds/4488
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-c/builds/3588
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-t/builds/3594
success: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/4503
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-c/builds/3591
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/3597
success: http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/4505
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-c/builds/3591
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-t/builds/3596
success: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android/builds/3665
success: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android-t/builds/1444
success: http://buildbot.rust-lang.org/builders/auto-win-32-opt/builds/4497
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-c/builds/3598
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/3604
success: http://buildbot.rust-lang.org/builders/auto-bsd-64-opt/builds/4264
82747ed
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.
fast-forwarding master to auto = 6c41f99