Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Refactor
Vec::map_in_place to move code out of PartialVec
Additionally, support zero-sized types.
- Loading branch information
454d91dThere 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 aturon
at tbu-@454d91d
454d91dThere 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 tbu-/rust/pr_mapinplace2 = 454d91d into auto
454d91dThere 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.
tbu-/rust/pr_mapinplace2 = 454d91d merged ok, testing candidate = f28df907
454d91dThere 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-nopt-t/builds/1607
exception: http://buildbot.rust-lang.org/builders/auto-mac-32-opt/builds/1617
exception: http://buildbot.rust-lang.org/builders/auto-mac-64-opt/builds/1614
exception: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-c/builds/1608
exception: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-t/builds/1614
exception: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/1611
exception: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-c/builds/1610
exception: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/1610
exception: http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/1614
exception: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-c/builds/1608
exception: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android-t/builds/1610
exception: http://buildbot.rust-lang.org/builders/auto-win-32-opt/builds/1265
exception: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/1263
exception: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-c/builds/1267
exception: http://buildbot.rust-lang.org/builders/auto-win-64-opt/builds/100
exception: http://buildbot.rust-lang.org/builders/auto-win-64-nopt-t/builds/98
exception: http://buildbot.rust-lang.org/builders/auto-win-64-nopt-c/builds/98
454d91dThere 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 aturon
at tbu-@454d91d
454d91dThere 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 tbu-/rust/pr_mapinplace2 = 454d91d into auto
454d91dThere 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.
tbu-/rust/pr_mapinplace2 = 454d91d merged ok, testing candidate = d53874e
454d91dThere 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/1620
success: http://buildbot.rust-lang.org/builders/auto-mac-64-opt/builds/1617
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-c/builds/1611
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-t/builds/1617
success: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/1614
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-c/builds/1613
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/1613
success: http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/1617
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-c/builds/1611
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-t/builds/1610
success: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android-t/builds/1613
success: http://buildbot.rust-lang.org/builders/auto-win-32-opt/builds/1268
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/1266
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-c/builds/1270
success: http://buildbot.rust-lang.org/builders/auto-win-64-opt/builds/103
success: http://buildbot.rust-lang.org/builders/auto-win-64-nopt-t/builds/101
success: http://buildbot.rust-lang.org/builders/auto-win-64-nopt-c/builds/101
454d91dThere 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 = d53874e