Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Add regression test for issue #17734
- Loading branch information
Showing
1 changed file
with
23 additions
and
0 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,23 @@ | ||
// Copyright 2014 The Rust Project Developers. See the COPYRIGHT | ||
// file at the top-level directory of this distribution and at | ||
// http://rust-lang.org/COPYRIGHT. | ||
// | ||
// Licensed under the Apache License, Version 2.0 <LICENSE-APACHE or | ||
// http://www.apache.org/licenses/LICENSE-2.0> or the MIT license | ||
// <LICENSE-MIT or http://opensource.org/licenses/MIT>, at your | ||
// option. This file may not be copied, modified, or distributed | ||
// except according to those terms. | ||
|
||
// Test that generating drop glue for Box<str> doesn't ICE | ||
|
||
fn f(s: Box<str>) -> Box<str> { | ||
s | ||
} | ||
|
||
fn main() { | ||
// There is currently no safe way to construct a `Box<str>`, so improvise | ||
let box_arr: Box<[u8]> = box ['h' as u8, 'e' as u8, 'l' as u8, 'l' as u8, 'o' as u8]; | ||
let box_str: Box<str> = unsafe { std::mem::transmute(box_arr) }; | ||
assert_eq!(box_str.as_slice(), "hello"); | ||
f(box_str); | ||
} |
714c8d8
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 alexcrichton
at bkoropoff@714c8d8
714c8d8
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 bkoropoff/rust/issue-17734 = 714c8d8 into auto
714c8d8
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.
bkoropoff/rust/issue-17734 = 714c8d8 merged ok, testing candidate = f56c1c9
714c8d8
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/1675
success: http://buildbot.rust-lang.org/builders/auto-mac-64-opt/builds/1672
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-c/builds/1666
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-t/builds/1672
success: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/1669
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-c/builds/1668
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/1668
success: http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/1672
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-c/builds/1666
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-t/builds/1665
success: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android-t/builds/1668
success: http://buildbot.rust-lang.org/builders/auto-win-32-opt/builds/1324
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/1322
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-c/builds/1326
success: http://buildbot.rust-lang.org/builders/auto-win-64-opt/builds/159
success: http://buildbot.rust-lang.org/builders/auto-win-64-nopt-t/builds/157
success: http://buildbot.rust-lang.org/builders/auto-win-64-nopt-c/builds/156
714c8d8
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 = f56c1c9