Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Don't double free embedded, unsized slices
- Loading branch information
Showing
2 changed files
with
15 additions
and
9 deletions.
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
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
415d7e8
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, nikomatsakis
at nrc@415d7e8
415d7e8
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 nick29581/rust/dst-bug-4 = 415d7e8 into auto
415d7e8
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.
nick29581/rust/dst-bug-4 = 415d7e8 merged ok, testing candidate = f35a796b
415d7e8
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-mac-64-opt/builds/1063
exception: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/1058
exception: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android-t/builds/1061
415d7e8
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, nikomatsakis
at nrc@415d7e8
415d7e8
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 nick29581/rust/dst-bug-4 = 415d7e8 into auto
415d7e8
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.
nick29581/rust/dst-bug-4 = 415d7e8 merged ok, testing candidate = 6025926
415d7e8
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/1065
success: http://buildbot.rust-lang.org/builders/auto-mac-64-opt/builds/1065
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-c/builds/1064
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-t/builds/1065
success: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/1060
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-c/builds/1061
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/1060
success: http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/1066
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-c/builds/1061
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-t/builds/1060
success: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android-t/builds/1063
success: http://buildbot.rust-lang.org/builders/auto-win-32-opt/builds/1064
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/1062
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-c/builds/1063
415d7e8
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 = 6025926