diff options
| author | Niko Matsakis <niko@alum.mit.edu> | 2014-10-21 23:48:39 -0400 |
|---|---|---|
| committer | Niko Matsakis <niko@alum.mit.edu> | 2014-10-21 23:52:00 -0400 |
| commit | aeba2ccf3082522f56e9aaefc0a961e2ed737f53 (patch) | |
| tree | b861d6e3a9d654a0a901889a79d49b3f3621a95b | |
| parent | 3d2cf60631502567007ea652f8ef299d907ee2c3 (diff) | |
| download | rust-aeba2ccf3082522f56e9aaefc0a961e2ed737f53.tar.gz rust-aeba2ccf3082522f56e9aaefc0a961e2ed737f53.zip | |
Adjust orphan rules to consider all input types, not just self type.
Fixes #18222.
| -rw-r--r-- | src/librustc/middle/traits/coherence.rs | 6 | ||||
| -rw-r--r-- | src/test/auxiliary/coherence-orphan-lib.rs | 14 | ||||
| -rw-r--r-- | src/test/compile-fail/coherence-orphan.rs | 25 |
3 files changed, 42 insertions, 3 deletions
diff --git a/src/librustc/middle/traits/coherence.rs b/src/librustc/middle/traits/coherence.rs index 6dc7fbf5d7c..e045b9fd4f4 100644 --- a/src/librustc/middle/traits/coherence.rs +++ b/src/librustc/middle/traits/coherence.rs @@ -67,9 +67,9 @@ pub fn impl_is_local(tcx: &ty::ctxt, return true; } - // Otherwise, self type must be local to the crate. - let self_ty = ty::lookup_item_type(tcx, impl_def_id).ty; - return ty_is_local(tcx, self_ty); + // Otherwise, at least one of the input types must be local to the + // crate. + trait_ref.input_types().iter().any(|&t| ty_is_local(tcx, t)) } pub fn ty_is_local(tcx: &ty::ctxt, diff --git a/src/test/auxiliary/coherence-orphan-lib.rs b/src/test/auxiliary/coherence-orphan-lib.rs new file mode 100644 index 00000000000..2e5d18b58f2 --- /dev/null +++ b/src/test/auxiliary/coherence-orphan-lib.rs @@ -0,0 +1,14 @@ +// 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. + +pub trait TheTrait<T> { + fn the_fn(&self); +} + diff --git a/src/test/compile-fail/coherence-orphan.rs b/src/test/compile-fail/coherence-orphan.rs new file mode 100644 index 00000000000..c44b0da5b15 --- /dev/null +++ b/src/test/compile-fail/coherence-orphan.rs @@ -0,0 +1,25 @@ +// 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. + +// aux-build:coherence-orphan-lib.rs + +extern crate "coherence-orphan-lib" as lib; + +use lib::TheTrait; + +struct TheType; + +impl TheTrait<uint> for int { } //~ ERROR E0117 + +impl TheTrait<TheType> for int { } + +impl TheTrait<int> for TheType { } + +fn main() { } |
