mirror of
https://github.com/rust-lang/rust.git
synced 2024-10-31 14:31:55 +00:00
borrowck: aliasability violation with closure captures can happen
It was considered to be impossible but actually it can happen for nested closures. Also, because there must be nested closures when this happens, we can use more targeted help message. Closes #21390 Closes #21600
This commit is contained in:
parent
a6a6fadbb9
commit
2c6440e2a2
@ -770,16 +770,7 @@ impl<'a, 'tcx> BorrowckCtxt<'a, 'tcx> {
|
||||
MutabilityViolation => {
|
||||
"cannot assign to data"
|
||||
}
|
||||
BorrowViolation(euv::ClosureCapture(_)) => {
|
||||
// I don't think we can get aliasability violations
|
||||
// with closure captures, so no need to come up with a
|
||||
// good error message. The reason this cannot happen
|
||||
// is because we only capture local variables in
|
||||
// closures, and those are never aliasable.
|
||||
self.tcx.sess.span_bug(
|
||||
span,
|
||||
"aliasability violation with closure");
|
||||
}
|
||||
BorrowViolation(euv::ClosureCapture(_)) |
|
||||
BorrowViolation(euv::OverloadedOperator) |
|
||||
BorrowViolation(euv::AddrOf) |
|
||||
BorrowViolation(euv::AutoRef) |
|
||||
@ -809,9 +800,18 @@ impl<'a, 'tcx> BorrowckCtxt<'a, 'tcx> {
|
||||
self.tcx.sess.span_err(span,
|
||||
format!("{} in a captured outer \
|
||||
variable in an `Fn` closure", prefix).as_slice());
|
||||
if let BorrowViolation(euv::ClosureCapture(_)) = kind {
|
||||
// The aliasability violation with closure captures can
|
||||
// happen for nested closures, so we know the enclosing
|
||||
// closure incorrectly accepts an `Fn` while it needs to
|
||||
// be `FnMut`.
|
||||
span_help!(self.tcx.sess, self.tcx.map.span(id),
|
||||
"consider changing this to accept closures that implement `FnMut`");
|
||||
} else {
|
||||
span_help!(self.tcx.sess, self.tcx.map.span(id),
|
||||
"consider changing this closure to take self by mutable reference");
|
||||
}
|
||||
}
|
||||
mc::AliasableStatic(..) |
|
||||
mc::AliasableStaticMut(..) => {
|
||||
self.tcx.sess.span_err(
|
||||
|
27
src/test/compile-fail/issue-21600.rs
Normal file
27
src/test/compile-fail/issue-21600.rs
Normal file
@ -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 call_it<F>(f: F) where F: Fn() { f(); }
|
||||
|
||||
struct A;
|
||||
|
||||
impl A {
|
||||
fn gen(&self) {}
|
||||
fn gen_mut(&mut self) {}
|
||||
}
|
||||
|
||||
fn main() {
|
||||
let mut x = A;
|
||||
call_it(|| { //~ HELP consider changing this to accept closures that implement `FnMut`
|
||||
call_it(|| x.gen());
|
||||
call_it(|| x.gen_mut()); //~ ERROR cannot borrow data mutably in a captured outer
|
||||
//~^ ERROR cannot borrow data mutably in a captured outer
|
||||
});
|
||||
}
|
Loading…
Reference in New Issue
Block a user