Skip to content

Commit

Permalink
Feature gate associated type defaults
Browse files Browse the repository at this point in the history
There are multiple issues with them as designed and implemented.

cc #27364
  • Loading branch information
brson committed Jul 29, 2015
1 parent 1c8194c commit a09bce1
Show file tree
Hide file tree
Showing 6 changed files with 30 additions and 1 deletion.
1 change: 1 addition & 0 deletions src/libcore/lib.rs
Expand Up @@ -65,6 +65,7 @@
#![allow(raw_pointer_derive)]
#![deny(missing_docs)]

#![feature(associated_type_defaults)]
#![feature(intrinsics)]
#![feature(lang_items)]
#![feature(on_unimplemented)]
Expand Down
9 changes: 8 additions & 1 deletion src/libsyntax/feature_gate.rs
Expand Up @@ -165,7 +165,10 @@ const KNOWN_FEATURES: &'static [(&'static str, &'static str, Status)] = &[
("static_recursion", "1.3.0", Active),

// Allows default type parameters to influence type inference.
("default_type_parameter_fallback", "1.3.0", Active)
("default_type_parameter_fallback", "1.3.0", Active),

// Allows associated type defaults
("associated_type_defaults", "1.2.0", Active),
];
// (changing above list without updating src/doc/reference.md makes @cmr sad)

Expand Down Expand Up @@ -763,6 +766,10 @@ impl<'a, 'v> Visitor<'v> for PostExpansionVisitor<'a> {
self.gate_feature("const_fn", ti.span, "const fn is unstable");
}
}
ast::TypeTraitItem(_, Some(_)) => {
self.gate_feature("associated_type_defaults", ti.span,
"associated type defaults are unstable");
}
_ => {}
}
visit::walk_trait_item(self, ti);
Expand Down
2 changes: 2 additions & 0 deletions src/test/auxiliary/xcrate_associated_type_defaults.rs
Expand Up @@ -8,6 +8,8 @@
// option. This file may not be copied, modified, or distributed
// except according to those terms.

#![feature(associated_type_defaults)]

pub trait Foo {
type Input = usize;
fn bar(&self, _: Self::Input) {}
Expand Down
15 changes: 15 additions & 0 deletions src/test/compile-fail/feature-gate-assoc-type-defaults.rs
@@ -0,0 +1,15 @@
// 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.

trait Foo {
type Bar = u8; //~ ERROR associated type defaults are unstable
}

fn main() {}
2 changes: 2 additions & 0 deletions src/test/run-pass/default-associated-types.rs
Expand Up @@ -8,6 +8,8 @@
// option. This file may not be copied, modified, or distributed
// except according to those terms.

#![feature(associated_type_defaults)]

trait Foo<T> {
type Out = T;
fn foo(&self) -> Self::Out;
Expand Down
2 changes: 2 additions & 0 deletions src/test/run-pass/issue-25339.rs
Expand Up @@ -8,6 +8,8 @@
// option. This file may not be copied, modified, or distributed
// except according to those terms.

#![feature(associated_type_defaults)]

use std::marker::PhantomData;

pub trait Routing<I> {
Expand Down

0 comments on commit a09bce1

Please sign in to comment.