Skip to content

Commit 9f9ac89

Browse files
borsbors
committed
Auto merge of #52959 - matthewjasper:closure-spans, r=pnkfelix
[NLL] Use smaller spans for errors involving closure captures Closes #51170 Closes #46599 Error messages involving closures now point to the captured variable/closure args. r? @pnkfelix
2 parents b47c314 + 12af36a commit 9f9ac89

37 files changed

+1253
-330
lines changed

‎src/librustc_mir/borrow_check/error_reporting.rs‎

Lines changed: 359 additions & 160 deletions
Large diffs are not rendered by default.

‎src/librustc_mir/borrow_check/mod.rs‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1542,7 +1542,7 @@ impl<'cx, 'gcx, 'tcx> MirBorrowckCtxt<'cx, 'gcx, 'tcx> {
15421542
if borrow_of_local_data(&borrow.borrowed_place) {
15431543
let err = self.tcx
15441544
.cannot_borrow_across_generator_yield(
1545-
self.retrieve_borrow_span(borrow),
1545+
self.retrieve_borrow_spans(borrow).var_or_use(),
15461546
yield_span,
15471547
Origin::Mir,
15481548
);

‎src/librustc_mir/borrow_check/mutability_errors.rs‎

Lines changed: 17 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -160,7 +160,6 @@ impl<'a, 'gcx, 'tcx> MirBorrowckCtxt<'a, 'gcx, 'tcx> {
160160
let act;
161161
let acted_on;
162162

163-
164163
let span = match error_access {
165164
AccessKind::Move => {
166165
err = self.tcx
@@ -180,31 +179,23 @@ impl<'a, 'gcx, 'tcx> MirBorrowckCtxt<'a, 'gcx, 'tcx> {
180179
act = "borrow as mutable";
181180
acted_on = "borrowed as mutable";
182181

183-
let closure_span = self.find_closure_span(span, location);
184-
if let Some((args, var)) = closure_span {
185-
err = self.tcx.cannot_borrow_path_as_mutable_because(
186-
args,
187-
&item_msg,
188-
&reason,
189-
Origin::Mir,
190-
);
191-
err.span_label(
192-
var,
193-
format!(
194-
"mutable borrow occurs due to use of `{}` in closure",
195-
self.describe_place(access_place).unwrap(),
196-
),
197-
);
198-
args
199-
} else {
200-
err = self.tcx.cannot_borrow_path_as_mutable_because(
201-
span,
202-
&item_msg,
203-
&reason,
204-
Origin::Mir,
205-
);
206-
span
207-
}
182+
let borrow_spans = self.borrow_spans(span, location);
183+
let borrow_span = borrow_spans.args_or_use();
184+
err = self.tcx.cannot_borrow_path_as_mutable_because(
185+
borrow_span,
186+
&item_msg,
187+
&reason,
188+
Origin::Mir,
189+
);
190+
borrow_spans.var_span_label(
191+
&mut err,
192+
format!(
193+
"mutable borrow occurs due to use of `{}` in closure",
194+
// always Some() if the message is printed.
195+
self.describe_place(access_place).unwrap_or(String::new()),
196+
)
197+
);
198+
borrow_span
208199
}
209200
};
210201

‎src/librustc_mir/borrow_check/nll/explain_borrow/mod.rs‎

Lines changed: 17 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -62,18 +62,24 @@ impl<'cx, 'gcx, 'tcx> MirBorrowckCtxt<'cx, 'gcx, 'tcx> {
6262
);
6363

6464
match find_use::find(mir, regioncx, tcx, region_sub, context.loc) {
65-
Some(Cause::LiveVar(_local, location)) => {
66-
if self.is_borrow_location_in_loop(context.loc) {
67-
err.span_label(
68-
mir.source_info(location).span,
69-
"borrow used here in later iteration of loop".to_string(),
70-
);
65+
Some(Cause::LiveVar(local, location)) => {
66+
let span = mir.source_info(location).span;
67+
let spans = self.move_spans(&Place::Local(local), location)
68+
.or_else(|| self.borrow_spans(span, location));
69+
let message = if self.is_borrow_location_in_loop(context.loc) {
70+
if spans.for_closure() {
71+
"borrow captured here by closure in later iteration of loop"
72+
} else {
73+
"borrow used here in later iteration of loop"
74+
}
7175
} else {
72-
err.span_label(
73-
mir.source_info(location).span,
74-
"borrow later used here".to_string(),
75-
);
76-
}
76+
if spans.for_closure() {
77+
"borrow later captured here by closure"
78+
} else {
79+
"borrow later used here"
80+
}
81+
};
82+
err.span_label(spans.var_or_use(), message);
7783
}
7884

7985
Some(Cause::DropVar(local, location)) => match &mir.local_decls[local].name {

‎src/librustc_mir/util/borrowck_errors.rs‎

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -203,8 +203,15 @@ pub trait BorrowckErrors<'cx>: Sized + Copy {
203203
desc,
204204
OGN = o
205205
);
206-
err.span_label(old_loan_span, "first closure is constructed here");
207-
err.span_label(new_loan_span, "second closure is constructed here");
206+
if old_loan_span == new_loan_span {
207+
err.span_label(
208+
old_loan_span,
209+
"closures are constructed here in different iterations of loop"
210+
);
211+
} else {
212+
err.span_label(old_loan_span, "first closure is constructed here");
213+
err.span_label(new_loan_span, "second closure is constructed here");
214+
}
208215
if let Some(old_load_end_span) = old_load_end_span {
209216
err.span_label(old_load_end_span, "borrow from first closure ends here");
210217
}

‎src/test/ui/borrowck/borrowck-closures-two-mut.stderr‎

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -77,11 +77,11 @@ error[E0499]: cannot borrow `x` as mutable more than once at a time (Mir)
7777
--> $DIR/borrowck-closures-two-mut.rs:24:24
7878
|
7979
LL | let c1 = to_fn_mut(|| x = 4);
80-
| -- - previous borrow occurs due to use of `x` in closure
80+
| -- - first borrow occurs due to use of `x` in closure
8181
| |
8282
| first mutable borrow occurs here
8383
LL | let c2 = to_fn_mut(|| x = 5); //~ ERROR cannot borrow `x` as mutable more than once
84-
| ^^ - borrow occurs due to use of `x` in closure
84+
| ^^ - second borrow occurs due to use of `x` in closure
8585
| |
8686
| second mutable borrow occurs here
8787
LL | //~| ERROR cannot borrow `x` as mutable more than once
@@ -92,11 +92,11 @@ error[E0499]: cannot borrow `x` as mutable more than once at a time (Mir)
9292
--> $DIR/borrowck-closures-two-mut.rs:36:24
9393
|
9494
LL | let c1 = to_fn_mut(|| set(&mut x));
95-
| -- - previous borrow occurs due to use of `x` in closure
95+
| -- - first borrow occurs due to use of `x` in closure
9696
| |
9797
| first mutable borrow occurs here
9898
LL | let c2 = to_fn_mut(|| set(&mut x)); //~ ERROR cannot borrow `x` as mutable more than once
99-
| ^^ - borrow occurs due to use of `x` in closure
99+
| ^^ - second borrow occurs due to use of `x` in closure
100100
| |
101101
| second mutable borrow occurs here
102102
LL | //~| ERROR cannot borrow `x` as mutable more than once
@@ -107,11 +107,11 @@ error[E0499]: cannot borrow `x` as mutable more than once at a time (Mir)
107107
--> $DIR/borrowck-closures-two-mut.rs:44:24
108108
|
109109
LL | let c1 = to_fn_mut(|| x = 5);
110-
| -- - previous borrow occurs due to use of `x` in closure
110+
| -- - first borrow occurs due to use of `x` in closure
111111
| |
112112
| first mutable borrow occurs here
113113
LL | let c2 = to_fn_mut(|| set(&mut x)); //~ ERROR cannot borrow `x` as mutable more than once
114-
| ^^ - borrow occurs due to use of `x` in closure
114+
| ^^ - second borrow occurs due to use of `x` in closure
115115
| |
116116
| second mutable borrow occurs here
117117
LL | //~| ERROR cannot borrow `x` as mutable more than once
@@ -122,11 +122,11 @@ error[E0499]: cannot borrow `x` as mutable more than once at a time (Mir)
122122
--> $DIR/borrowck-closures-two-mut.rs:52:24
123123
|
124124
LL | let c1 = to_fn_mut(|| x = 5);
125-
| -- - previous borrow occurs due to use of `x` in closure
125+
| -- - first borrow occurs due to use of `x` in closure
126126
| |
127127
| first mutable borrow occurs here
128128
LL | let c2 = to_fn_mut(|| { let _y = to_fn_mut(|| set(&mut x)); }); // (nested closure)
129-
| ^^ - borrow occurs due to use of `x` in closure
129+
| ^^ - second borrow occurs due to use of `x` in closure
130130
| |
131131
| second mutable borrow occurs here
132132
...
@@ -137,11 +137,11 @@ error[E0499]: cannot borrow `x` as mutable more than once at a time (Mir)
137137
--> $DIR/borrowck-closures-two-mut.rs:65:24
138138
|
139139
LL | let c1 = to_fn_mut(|| set(&mut *x.f));
140-
| -- - previous borrow occurs due to use of `x` in closure
140+
| -- - first borrow occurs due to use of `x` in closure
141141
| |
142142
| first mutable borrow occurs here
143143
LL | let c2 = to_fn_mut(|| set(&mut *x.f));
144-
| ^^ - borrow occurs due to use of `x` in closure
144+
| ^^ - second borrow occurs due to use of `x` in closure
145145
| |
146146
| second mutable borrow occurs here
147147
...

‎src/test/ui/borrowck/borrowck-escaping-closure-error-1.nll.stderr‎

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,10 @@
11
error[E0597]: `books` does not live long enough
2-
--> $DIR/borrowck-escaping-closure-error-1.rs:23:11
2+
--> $DIR/borrowck-escaping-closure-error-1.rs:23:14
33
|
44
LL | spawn(|| books.push(4));
5-
| ^^^^^^^^^^^^^^^^ borrowed value does not live long enough
5+
| -- ^^^^^ borrowed value does not live long enough
6+
| |
7+
| value captured here
68
LL | //~^ ERROR E0373
79
LL | }
810
| - `books` dropped here while still borrowed

‎src/test/ui/borrowck/borrowck-escaping-closure-error-2.nll.stderr‎

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,10 @@
11
error[E0597]: `books` does not live long enough
2-
--> $DIR/borrowck-escaping-closure-error-2.rs:21:14
2+
--> $DIR/borrowck-escaping-closure-error-2.rs:21:17
33
|
44
LL | Box::new(|| books.push(4))
5-
| ^^^^^^^^^^^^^^^^ borrowed value does not live long enough
5+
| -- ^^^^^ borrowed value does not live long enough
6+
| |
7+
| value captured here
68
LL | //~^ ERROR E0373
79
LL | }
810
| - `books` dropped here while still borrowed

‎src/test/ui/error-codes/E0504.nll.stderr‎

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,17 +1,16 @@
11
error[E0505]: cannot move out of `fancy_num` because it is borrowed
22
--> $DIR/E0504.rs:19:13
33
|
4-
LL | let fancy_ref = &fancy_num;
5-
| ---------- borrow of `fancy_num` occurs here
4+
LL | let fancy_ref = &fancy_num;
5+
| ---------- borrow of `fancy_num` occurs here
66
LL |
7-
LL | let x = move || {
8-
| _____________^
9-
LL | | println!("child function: {}", fancy_num.num); //~ ERROR E0504
10-
LL | | };
11-
| |_____^ move out of `fancy_num` occurs here
7+
LL | let x = move || {
8+
| ^^^^^^^ move out of `fancy_num` occurs here
9+
LL | println!("child function: {}", fancy_num.num); //~ ERROR E0504
10+
| --------- move occurs due to use in closure
1211
...
13-
LL | println!("main function: {}", fancy_ref.num);
14-
| ------------- borrow later used here
12+
LL | println!("main function: {}", fancy_ref.num);
13+
| ------------- borrow later used here
1514

1615
error: aborting due to previous error
1716

‎src/test/ui/issue-11192.nll.stderr‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ LL | let mut test = |foo: &Foo| {
55
| ----------- mutable borrow occurs here
66
LL | println!("access {}", foo.x);
77
LL | ptr = box Foo { x: ptr.x + 1 };
8-
| --- previous borrow occurs due to use of `ptr` in closure
8+
| --- first borrow occurs due to use of `ptr` in closure
99
...
1010
LL | test(&*ptr);
1111
| -----^^^^^-

0 commit comments

Comments
 (0)