Do not permit type parameters on builtin bounds.

Author: ahmedcharles

src/librustc_typeck/astconv.rs

@@ -1847,8 +1847,14 @@ pub fn partition_bounds<'a>(tcx: &ty::ctxt,
                         if ty::try_add_builtin_trait(tcx,
                                                      trait_did,
                                                      &mut builtin_bounds) {
-                            // FIXME(#20302) -- we should check for things like Copy<T>
-                            continue; // success
+                            let segments = &b.trait_ref.path.segments;
+                            let parameters = &segments[segments.len() - 1].parameters;
+                            if parameters.is_empty() {
+                                continue; // success
+                            }
+                            span_err!(tcx.sess, b.trait_ref.path.span, E0316,
+                                      "builtin bounds do not require arguments, {} given",
+                                      parameters.types().len());
                         }
                     }
                     _ => {

src/librustc_typeck/diagnostics.rs

@@ -171,7 +171,8 @@ register_diagnostics! {
     E0247, // found module name used as a type
     E0248, // found value name used as a type
     E0249, // expected constant expr for array length
-    E0250  // expected constant expr for array length
+    E0250, // expected constant expr for array length
+    E0316  // wrong number of type arguments to a built-in trait
 }
 
 __build_diagnostic_array! { DIAGNOSTICS }

src/test/compile-fail/typeck-builtin-bound-type-parameters.rs

@@ -0,0 +1,27 @@
+// Copyright 2015 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.
+
+fn foo1<T:Copy<U>, U>(x: T) {}
+//~^ ERROR: builtin bounds do not require arguments, 1 given
+
+trait Trait: Copy<Send> {}
+//~^ ERROR: builtin bounds do not require arguments, 1 given
+
+struct MyStruct1<T: Copy<T>>;
+//~^ ERROR: builtin bounds do not require arguments, 1 given
+
+struct MyStruct2<'a, T: Copy<'a>>;
+//~^ ERROR: builtin bounds do not require arguments, 1 given
+
+fn foo2<'a, T:Copy<'a, U>, U>(x: T) {}
+//~^ ERROR: builtin bounds do not require arguments, 1 given
+
+fn main() {
+}