From aeba2ccf3082522f56e9aaefc0a961e2ed737f53 Mon Sep 17 00:00:00 2001 From: Niko Matsakis Date: Tue, 21 Oct 2014 23:48:39 -0400 Subject: [PATCH] Adjust orphan rules to consider all input types, not just self type. Fixes #18222. --- src/librustc/middle/traits/coherence.rs | 6 +++--- src/test/auxiliary/coherence-orphan-lib.rs | 14 ++++++++++++ src/test/compile-fail/coherence-orphan.rs | 25 ++++++++++++++++++++++ 3 files changed, 42 insertions(+), 3 deletions(-) create mode 100644 src/test/auxiliary/coherence-orphan-lib.rs create mode 100644 src/test/compile-fail/coherence-orphan.rs 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 or the MIT license +// , at your +// option. This file may not be copied, modified, or distributed +// except according to those terms. + +pub trait TheTrait { + 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 or the MIT license +// , 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 for int { } //~ ERROR E0117 + +impl TheTrait for int { } + +impl TheTrait for TheType { } + +fn main() { }