]> git.lizzy.rs Git - rust.git/commitdiff
Add kind-checking for assign-op, copy, ret, be, fail exprs. Fix caught kinding-violat...
authorGraydon Hoare <graydon@mozilla.com>
Tue, 23 Aug 2011 22:58:53 +0000 (15:58 -0700)
committerGraydon Hoare <graydon@mozilla.com>
Tue, 23 Aug 2011 22:58:53 +0000 (15:58 -0700)
28 files changed:
src/comp/driver/rustc.rs
src/comp/front/attr.rs
src/comp/front/test.rs
src/comp/middle/kind.rs
src/comp/syntax/ast_util.rs
src/comp/syntax/parse/parser.rs
src/comp/syntax/util/interner.rs
src/lib/task.rs
src/lib/unsafe.rs
src/lib/util.rs
src/test/compile-fail/block-copy.rs
src/test/compiletest/procsrv.rs
src/test/run-pass/auto-instantiate.rs
src/test/run-pass/autobind.rs
src/test/run-pass/box-unbox.rs
src/test/run-pass/fixed-point-bind-box.rs
src/test/run-pass/generic-alias-box.rs
src/test/run-pass/generic-bind-2.rs
src/test/run-pass/generic-bind.rs
src/test/run-pass/generic-derived-type.rs
src/test/run-pass/generic-fn-infer.rs
src/test/run-pass/generic-fn.rs
src/test/run-pass/generic-obj-with-derived-type.rs
src/test/run-pass/generic-obj.rs
src/test/run-pass/generic-tup.rs
src/test/run-pass/issue-333.rs
src/test/run-pass/newtype-polymorphic.rs
src/test/run-pass/ret-none.rs

index a1981e1d2832dae6fce5bbc89cf31410e62652f8..ded36db2cf8cad7cfe9ddb85367ec1408277d9c4 100644 (file)
@@ -108,7 +108,7 @@ fn parse_input_src(sess: session::session, cfg: &ast::crate_cfg, infile: str)
     ret {crate: crate, src: src};
 }
 
-fn time<T>(do_it: bool, what: str, thunk: fn() -> T) -> T {
+fn time<@T>(do_it: bool, what: str, thunk: fn() -> T) -> T {
     if !do_it { ret thunk(); }
     let start = std::time::precise_time_s();
     let rv = thunk();
index e199d10fbd18c9c3f6da5db6331528e5bc3fb4a0..5701ff33f975ac9d8bd88a61cc0f70b58d2d4222 100644 (file)
@@ -189,7 +189,7 @@ fn require_unique_names(sess: &session::session, metas: &[@ast::meta_item]) {
     }
 }
 
-fn span<T>(item: &T) -> ast::spanned<T> {
+fn span<@T>(item: &T) -> ast::spanned<T> {
     ret {node: item, span: ast_util::dummy_sp()};
 }
 
index d9cda6e601ad9bd27744c1d4e06e0c5a38e14d7d..8e0b71caf848d9581a63a0371e1df3e214ee7e56 100644 (file)
@@ -170,7 +170,7 @@ fn mk_test_module(cx: &test_ctxt) -> @ast::item {
     ret @item;
 }
 
-fn nospan<T>(t: &T) -> ast::spanned<T> {
+fn nospan<@T>(t: &T) -> ast::spanned<T> {
     ret {node: t, span: dummy_sp()};
 }
 
index 4ffdcc95ff3fd4bffa4d18d647224684d25fc86f..9a0d251e44e9aed98d14b5692b4fea42266f3449 100644 (file)
@@ -76,6 +76,7 @@
 import syntax::visit;
 
 import std::vec;
+import std::option;
 
 import ast::kind;
 import ast::kind_unique;
@@ -135,7 +136,20 @@ fn check_expr(tcx: &ty::ctxt, e: &@ast::expr) {
     alt e.node {
       ast::expr_move(a, b) { need_shared_lhs_rhs(tcx, a, b, "<-"); }
       ast::expr_assign(a, b) { need_shared_lhs_rhs(tcx, a, b, "="); }
+      ast::expr_assign_op(_, a, b) { need_shared_lhs_rhs(tcx, a, b, "op="); }
       ast::expr_swap(a, b) { need_shared_lhs_rhs(tcx, a, b, "<->"); }
+      ast::expr_copy(a) {
+        need_expr_kind(tcx, a, ast::kind_shared, "'copy' operand");
+      }
+      ast::expr_ret(option::some(a)) {
+        need_expr_kind(tcx, a, ast::kind_shared, "'ret' operand");
+      }
+      ast::expr_be(a) {
+        need_expr_kind(tcx, a, ast::kind_shared, "'be' operand");
+      }
+      ast::expr_fail(option::some(a)) {
+        need_expr_kind(tcx, a, ast::kind_shared, "'fail' operand");
+      }
       ast::expr_call(callee, _) {
         let tpt = ty::expr_ty_params_and_ty(tcx, callee);
 
index e3b16702b7624f7a7992527aba03a253f0bba14e..e89889fa0276f093f6c9ad11627d65db42c8108d 100644 (file)
@@ -3,7 +3,7 @@
 import codemap::span;
 import ast::*;
 
-fn respan<T>(sp: &span, t: &T) -> spanned<T> { ret {node: t, span: sp}; }
+fn respan<@T>(sp: &span, t: &T) -> spanned<T> { ret {node: t, span: sp}; }
 
 /* assuming that we're not in macro expansion */
 fn mk_sp(lo: uint, hi: uint) -> span {
index 495d1372d79bfa669fb88fdb0e3ebefbd32dbf21..aa553e32e3944960764826669c6dc30932599010 100644 (file)
@@ -222,7 +222,7 @@ fn expect_gt(p: &parser) {
     }
 }
 
-fn spanned<T>(lo: uint, hi: uint, node: &T) -> spanned<T> {
+fn spanned<@T>(lo: uint, hi: uint, node: &T) -> spanned<T> {
     ret {node: node, span: ast_util::mk_sp(lo, hi)};
 }
 
index 31656886a453197d29a5cd132f1b268cce0d8bc0..3d1a495fb2ee360f3d712a5793807e5a92193c93 100644 (file)
@@ -33,7 +33,7 @@ fn intern<@T>(itr: &interner<T>, val: &T) -> uint {
     }
 }
 
-fn get<T>(itr: &interner<T>, idx: uint) -> T { ret itr.vect[idx]; }
+fn get<@T>(itr: &interner<T>, idx: uint) -> T { ret itr.vect[idx]; }
 
 fn len<T>(itr: &interner<T>) -> uint { ret vec::len(itr.vect); }
 
index 2e88f12f6c0fdbd0bd4256decb868cf98aecfb53..52181528b4fd92ab04d47de10a9dff3b2bc6e07f 100644 (file)
 
 resource rust_task_ptr(task: *rust_task) { rustrt::drop_task(task); }
 
-fn get_task_ptr(id: task) -> rust_task_ptr {
-    ret rust_task_ptr(rustrt::get_task_pointer(id));
-}
-
 type task = int;
 type task_id = task;
 
@@ -107,7 +103,7 @@ fn spawn_inner(thunk: -fn(), notify: option<comm::chan<task_notification>>) ->
     // stack.
 
     // set up the task pointer
-    let task_ptr = get_task_ptr(id);
+    let task_ptr = rust_task_ptr(rustrt::get_task_pointer(id));
     let regs = ptr::addr_of((**task_ptr).ctx.regs);
     (*regs).edx = cast(*task_ptr);;
     (*regs).esp = cast((**task_ptr).stack_ptr);
index 6e1b4b3b7c3bbc74a2a281b2d8d086e5fbb1030c..0d382655b4191907fca91c15cefb2f0c4fdc48e8 100644 (file)
@@ -9,7 +9,7 @@
 }
 
 // Casts the value at `src` to U. The two types must have the same length.
-fn reinterpret_cast<T, U>(src: &T) -> U { ret rusti::cast(src); }
+fn reinterpret_cast<T, @U>(src: &T) -> U { ret rusti::cast(src); }
 
 fn leak<@T>(thing: -T) {
     rustrt::leak(thing);
index 24a14b1fa9210cd4c014db0e6ee4ff534ed5ecf2..6041da4d0d0d801ed78c0444fcd191c82a29829b 100644 (file)
@@ -1,6 +1,6 @@
 
 
-fn id<T>(x: &T) -> T { ret x; }
+fn id<@T>(x: &T) -> T { ret x; }
 
 
 /* FIXME (issue #141):  See test/run-pass/constrained-type.rs.  Uncomment
index 6a6c708cf8813ff81e3df40e7f003f9f81057930..76b05792f2fc735a8330d44de5fd6a4d44f5dd10 100644 (file)
@@ -1,4 +1,4 @@
-// error-pattern: non-copyable
+// error-pattern: mismatched kinds
 
 fn lol(f: &block()) -> block() { ret f; }
 fn main() { let i = 8; let f = lol(block () { log_err i; }); f(); }
index 6e464038b185830c8bfe4059b7297604864d2406..efbc26f669a5b87120d13b3c6921a3400248f084 100644 (file)
@@ -150,7 +150,7 @@ fn worker(p: port<request>) {
     }
 }
 
-fn with_lib_path<T>(path: &str, f: fn() -> T) -> T {
+fn with_lib_path<@T>(path: &str, f: fn() -> T) -> T {
     let maybe_oldpath = getenv(util::lib_path_env_var());
     append_lib_path(path);
     let res = f();
index f8d9ebbb587edf903eb852f2725118177875aa3f..e06da60789c3ca18364d5055c0dba0b8b7bac598 100644 (file)
@@ -2,6 +2,6 @@
 
 
 // -*- rust -*-
-fn f<T, U>(x: &T, y: &U) -> {a: T, b: U} { ret {a: x, b: y}; }
+fn f<@T, @U>(x: &T, y: &U) -> {a: T, b: U} { ret {a: x, b: y}; }
 
 fn main() { log f({x: 3, y: 4, z: 5}, 4).a.x; log f(5, 6).a; }
index 536db525d5c93d00b92b62dba5fa3beceb43c782..ebf2bd432d4e81849260cccfd22adf5bd496a568 100644 (file)
@@ -1,4 +1,4 @@
-fn f<T>(x: &[T]) -> T { ret x[0]; }
+fn f<@T>(x: &[T]) -> T { ret x[0]; }
 
 fn g(act: fn(&[int]) -> int) -> int { ret act([1, 2, 3]); }
 
index e27954d4d655e8ae621f29f460d84c724621a30f..08633a572a3ebcb97df3bc48a6936fb41356181b 100644 (file)
@@ -2,7 +2,7 @@
 
 type box<T> = {c: @T};
 
-fn unbox<T>(b: &box<T>) -> T { ret *b.c; }
+fn unbox<@T>(b: &box<T>) -> T { ret *b.c; }
 
 fn main() {
     let foo: int = 17;
index 4ca1cdb34c9d514ce721c35a5bc61f6198aca0e9..57dcc98b2a86fe7486f8ff485983315f362d079a 100644 (file)
@@ -1,8 +1,8 @@
-fn fix_help<A, B>(f: @fn(@fn(&A) -> B, &A) -> B, x: &A) -> B {
+fn fix_help<A, @B>(f: @fn(@fn(&A) -> B, &A) -> B, x: &A) -> B {
     ret f(@bind fix_help(f, _), x);
 }
 
-fn fix<A, B>(f: @fn(@fn(&A) -> B, &A) -> B) -> @fn(&A) -> B {
+fn fix<A, @B>(f: @fn(@fn(&A) -> B, &A) -> B) -> @fn(&A) -> B {
     ret @bind fix_help(f, _);
 }
 
index 8d58c580fba5c022bb58d860390f05776546fbe1..3250b88ab824f6c69a508c8c372142315788dd38 100644 (file)
@@ -1,6 +1,6 @@
 
 
-fn id<T>(t: &T) -> T { ret t; }
+fn id<@T>(t: &T) -> T { ret t; }
 
 fn main() {
     let expected = @100;
index 499bc2a25d1fafb6e444f32fb0bdbb30637ba046..5c149ad59360c279f1b5b2e5c46346f87fb5334e 100644 (file)
@@ -1,6 +1,6 @@
 
 
-fn id<T>(t: &T) -> T { ret t; }
+fn id<@T>(t: &T) -> T { ret t; }
 
 fn main() {
     let t = {a: 1, b: 2, c: 3, d: 4, e: 5, f: 6, g: 7};
index 97b000ee1b52fcd3253037c85332337a98e28b9f..01156f579004b83b9fc0dac414df9cae16e26f4f 100644 (file)
@@ -1,6 +1,6 @@
 
 
-fn id<T>(t: &T) -> T { ret t; }
+fn id<@T>(t: &T) -> T { ret t; }
 
 fn main() {
     let t = {_0: 1, _1: 2, _2: 3, _3: 4, _4: 5, _5: 6, _6: 7};
index d0b98b1068e245b04a4564616e90050e3aaa147d..c11dce2ac7b46f99d11cc64e40c26ee4177447d2 100644 (file)
@@ -1,8 +1,8 @@
 
 
-fn g<X>(x: &X) -> X { ret x; }
+fn g<@X>(x: &X) -> X { ret x; }
 
-fn f<T>(t: &T) -> {a: T, b: T} {
+fn f<@T>(t: &T) -> {a: T, b: T} {
     type pair = {a: T, b: T};
 
     let x: pair = {a: t, b: t};
index 4c6597e97d759ede1ff7c4b43cd37d2f95cb2714..559fa4d04465408641b098099f6effc9c37131e0 100644 (file)
@@ -4,6 +4,6 @@
 // -*- rust -*-
 
 // Issue #45: infer type parameters in function applications
-fn id<T>(x: &T) -> T { ret x; }
+fn id<@T>(x: &T) -> T { ret x; }
 
 fn main() { let x: int = 42; let y: int = id(x); assert (x == y); }
index 5eb4e0aae328ece372f69a0e8ceb7b1236b8bb25..940e0fe5aed77e0c22608761e8339d02555fe9c2 100644 (file)
@@ -2,7 +2,7 @@
 
 
 // -*- rust -*-
-fn id<T>(x: &T) -> T { ret x; }
+fn id<@T>(x: &T) -> T { ret x; }
 
 type triple = {x: int, y: int, z: int};
 
index 05552db04ffa6ce8eef029efab87e8e36c99946e..0bc1fafa495520bb5ed587c299b70d621e58c332 100644 (file)
@@ -1,6 +1,6 @@
 
 
-obj handle<T>(data: T) {
+obj handle<@T>(data: T) {
     fn get() -> T { ret data; }
 }
 
index f22addc8b512a3e44cb7e3a72219d59652172c04..6653014b0246a3e2575ef95a3347833858c8bdd2 100644 (file)
@@ -1,6 +1,6 @@
 
 
-obj buf<T>(data: {_0: T, _1: T, _2: T}) {
+obj buf<@T>(data: {_0: T, _1: T, _2: T}) {
     fn get(i: int) -> T {
         if i == 0 {
             ret data._0;
index 8cacf51e9b51b4149e957ec888bc282a6751f22d..ac15998c1ccddf18750ffd9b08bad4ebcf8245e7 100644 (file)
@@ -1,4 +1,4 @@
-fn get_third<T>(t: &(T, T, T)) -> T { let (_, _, x) = t; ret x; }
+fn get_third<@T>(t: &(T, T, T)) -> T { let (_, _, x) = t; ret x; }
 
 fn main() {
     log get_third((1, 2, 3));
index 12afefe57b111850f0db61ceed30127f10a40c25..b5304234b987e3970d64e00a66fb867d25329f57 100644 (file)
@@ -1,5 +1,5 @@
-fn quux<T>(x: &T) -> T { let f = id::<T>; ret f(x); }
+fn quux<@T>(x: &T) -> T { let f = id::<T>; ret f(x); }
 
-fn id<T>(x: &T) -> T { ret x; }
+fn id<@T>(x: &T) -> T { ret x; }
 
 fn main() { assert (quux(10) == 10); }
index fc3002ba6627e5fcc8eee78023052d9f240b7cf4..27fbaee6a777646bd52deb7054b4143ae2770cd4 100644 (file)
@@ -1,8 +1,8 @@
 tag myvec<X> = [X];
 
-fn myvec_deref<X>(mv: &myvec<X>) -> [X] { ret *mv; }
+fn myvec_deref<@X>(mv: &myvec<X>) -> [X] { ret *mv; }
 
-fn myvec_elt<X>(mv: &myvec<X>) -> X { ret mv[0]; }
+fn myvec_elt<@X>(mv: &myvec<X>) -> X { ret mv[0]; }
 
 fn main() {
     let mv = myvec([1, 2, 3]);
index b56db9f9c700744ed3cc177d3fca4371ab815090..841753d7ad19236730f343ef5848e779a5e7afb9 100644 (file)
@@ -2,6 +2,6 @@
 
 tag option<T> { none; some(T); }
 
-fn f<T>() -> option<T> { ret none; }
+fn f<@T>() -> option<T> { ret none; }
 
 fn main() { f::<int>(); }