about summary refs log tree commit diff
diff options
context:
space:
mode:
authorbors <bors@rust-lang.org>2014-06-24 12:01:50 +0000
committerbors <bors@rust-lang.org>2014-06-24 12:01:50 +0000
commit71fe44def9676d519f5ce5d7304e581a42cf2c70 (patch)
treec4d183f7e5d701cae8f1f6249f0c2dd816abb6dc
parent58bf8b2155dc2ece0cee2bc2f60c72ea60d75af3 (diff)
parent7be2019428ff888ea26075a065fa3cebb66c5ead (diff)
downloadrust-71fe44def9676d519f5ce5d7304e581a42cf2c70.tar.gz
rust-71fe44def9676d519f5ce5d7304e581a42cf2c70.zip
auto merge of #15113 : pnkfelix/rust/fsk-add-regression-test-for-ice-from-10846, r=alexcrichton
Includes a bit more comments than usual for a regression test; I felt like documenting Niko's diagnosis of the original problem here. 

Fix #15111 

r? anyone.
-rw-r--r--src/test/run-pass/regions-lifetime-nonfree-late-bound.rs40
1 files changed, 40 insertions, 0 deletions
diff --git a/src/test/run-pass/regions-lifetime-nonfree-late-bound.rs b/src/test/run-pass/regions-lifetime-nonfree-late-bound.rs
new file mode 100644
index 00000000000..1c2e50a5f76
--- /dev/null
+++ b/src/test/run-pass/regions-lifetime-nonfree-late-bound.rs
@@ -0,0 +1,40 @@
+// 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.
+
+// This is a regression test for the ICE from issue #10846.
+//
+// The original issue causing the ICE: the LUB-computations during
+// type inference were encountering late-bound lifetimes, and
+// asserting that such lifetimes should have already been subsituted
+// with a concrete lifetime.
+//
+// However, those encounters were occurring within the lexical scope
+// of the binding for the late-bound lifetime; that is, the late-bound
+// lifetimes were perfectly valid.  The core problem was that the type
+// folding code was over-zealously passing back all lifetimes when
+// doing region-folding, when really all clients of the region-folding
+// case only want to see FREE lifetime variables, not bound ones.
+
+pub fn main() {
+    fn explicit() {
+        fn test(_x: Option<|f: <'a> |g: &'a int||>) {}
+        test(Some(|_f: <'a> |g: &'a int|| {}));
+    }
+
+    // The code below is shorthand for the code above (and more likely
+    // to represent what one encounters in practice).
+    fn implicit() {
+        fn test(_x: Option<|f:      |g: &   int||>) {}
+        test(Some(|_f:      |g: &   int|| {}));
+    }
+
+    explicit();
+    implicit();
+}