2018-08-01 14:40:06 -05:00
|
|
|
error[E0505]: cannot move out of `x` because it is borrowed
|
2019-04-07 10:07:36 -05:00
|
|
|
--> $DIR/closure-borrow-spans.rs:5:13
|
2018-08-01 14:40:06 -05:00
|
|
|
|
|
|
|
|
LL | let f = || x.len();
|
|
|
|
| -- - borrow occurs due to use in closure
|
|
|
|
| |
|
|
|
|
| borrow of `x` occurs here
|
2019-03-09 06:03:44 -06:00
|
|
|
LL | let y = x;
|
2018-08-01 14:40:06 -05:00
|
|
|
| ^ move out of `x` occurs here
|
|
|
|
LL | f.use_ref();
|
Use larger span for adjustments on method calls
Currently, we use a relatively 'small' span for THIR
expressions generated by an 'adjustment' (e.g. an autoderef,
autoborrow, unsizing). As a result, if a borrow generated
by an adustment ends up causing a borrowcheck error, for example:
```rust
let mut my_var = String::new();
let my_ref = &my_var
my_var.push('a');
my_ref;
```
then the span for the mutable borrow may end up referring
to only the base expression (e.g. `my_var`), rather than
the method call which triggered the mutable borrow
(e.g. `my_var.push('a')`)
Due to a quirk of the MIR borrowck implementation,
this doesn't always get exposed in migration mode,
but it does in many cases.
This commit makes THIR building consistently use 'larger'
spans for adjustment expressions
The intent of this change it make it clearer to users
when it's the specific way in which a variable is
used (for example, in a method call) that produdes
a borrowcheck error. For example, an error message
claiming that a 'mutable borrow occurs here' might
be confusing if it just points at a usage of a variable
(e.g. `my_var`), when no `&mut` is in sight. Pointing
at the entire expression should help to emphasize
that the method call itself is responsible for
the mutable borrow.
In several cases, this makes the `#![feature(nll)]` diagnostic
output match up exactly with the default (migration mode) output.
As a result, several `.nll.stderr` files end up getting removed
entirely.
2021-09-16 15:01:22 -05:00
|
|
|
| ----------- borrow later used here
|
2018-08-01 14:40:06 -05:00
|
|
|
|
|
|
|
error[E0502]: cannot borrow `x` as mutable because it is also borrowed as immutable
|
2019-04-07 10:07:36 -05:00
|
|
|
--> $DIR/closure-borrow-spans.rs:11:13
|
2018-08-01 14:40:06 -05:00
|
|
|
|
|
|
|
|
LL | let f = || x;
|
|
|
|
| -- - first borrow occurs due to use of `x` in closure
|
|
|
|
| |
|
|
|
|
| immutable borrow occurs here
|
2019-03-09 06:03:44 -06:00
|
|
|
LL | let y = &mut x;
|
2018-08-01 14:40:06 -05:00
|
|
|
| ^^^^^^ mutable borrow occurs here
|
|
|
|
LL | f.use_ref();
|
Use larger span for adjustments on method calls
Currently, we use a relatively 'small' span for THIR
expressions generated by an 'adjustment' (e.g. an autoderef,
autoborrow, unsizing). As a result, if a borrow generated
by an adustment ends up causing a borrowcheck error, for example:
```rust
let mut my_var = String::new();
let my_ref = &my_var
my_var.push('a');
my_ref;
```
then the span for the mutable borrow may end up referring
to only the base expression (e.g. `my_var`), rather than
the method call which triggered the mutable borrow
(e.g. `my_var.push('a')`)
Due to a quirk of the MIR borrowck implementation,
this doesn't always get exposed in migration mode,
but it does in many cases.
This commit makes THIR building consistently use 'larger'
spans for adjustment expressions
The intent of this change it make it clearer to users
when it's the specific way in which a variable is
used (for example, in a method call) that produdes
a borrowcheck error. For example, an error message
claiming that a 'mutable borrow occurs here' might
be confusing if it just points at a usage of a variable
(e.g. `my_var`), when no `&mut` is in sight. Pointing
at the entire expression should help to emphasize
that the method call itself is responsible for
the mutable borrow.
In several cases, this makes the `#![feature(nll)]` diagnostic
output match up exactly with the default (migration mode) output.
As a result, several `.nll.stderr` files end up getting removed
entirely.
2021-09-16 15:01:22 -05:00
|
|
|
| ----------- immutable borrow later used here
|
2018-08-01 14:40:06 -05:00
|
|
|
|
|
|
|
error[E0597]: `x` does not live long enough
|
2019-04-07 10:07:36 -05:00
|
|
|
--> $DIR/closure-borrow-spans.rs:19:16
|
2018-08-01 14:40:06 -05:00
|
|
|
|
|
2019-03-09 06:03:44 -06:00
|
|
|
LL | f = || x;
|
2018-08-01 14:40:06 -05:00
|
|
|
| -- ^ borrowed value does not live long enough
|
|
|
|
| |
|
|
|
|
| value captured here
|
|
|
|
LL | }
|
|
|
|
| - `x` dropped here while still borrowed
|
|
|
|
LL | f.use_ref();
|
Use larger span for adjustments on method calls
Currently, we use a relatively 'small' span for THIR
expressions generated by an 'adjustment' (e.g. an autoderef,
autoborrow, unsizing). As a result, if a borrow generated
by an adustment ends up causing a borrowcheck error, for example:
```rust
let mut my_var = String::new();
let my_ref = &my_var
my_var.push('a');
my_ref;
```
then the span for the mutable borrow may end up referring
to only the base expression (e.g. `my_var`), rather than
the method call which triggered the mutable borrow
(e.g. `my_var.push('a')`)
Due to a quirk of the MIR borrowck implementation,
this doesn't always get exposed in migration mode,
but it does in many cases.
This commit makes THIR building consistently use 'larger'
spans for adjustment expressions
The intent of this change it make it clearer to users
when it's the specific way in which a variable is
used (for example, in a method call) that produdes
a borrowcheck error. For example, an error message
claiming that a 'mutable borrow occurs here' might
be confusing if it just points at a usage of a variable
(e.g. `my_var`), when no `&mut` is in sight. Pointing
at the entire expression should help to emphasize
that the method call itself is responsible for
the mutable borrow.
In several cases, this makes the `#![feature(nll)]` diagnostic
output match up exactly with the default (migration mode) output.
As a result, several `.nll.stderr` files end up getting removed
entirely.
2021-09-16 15:01:22 -05:00
|
|
|
| ----------- borrow later used here
|
2018-08-01 14:40:06 -05:00
|
|
|
|
|
|
|
error[E0506]: cannot assign to `x` because it is borrowed
|
2019-04-07 10:07:36 -05:00
|
|
|
--> $DIR/closure-borrow-spans.rs:26:5
|
2018-08-01 14:40:06 -05:00
|
|
|
|
|
|
|
|
LL | let f = || x;
|
|
|
|
| -- - borrow occurs due to use in closure
|
|
|
|
| |
|
2023-01-14 21:06:44 -06:00
|
|
|
| `x` is borrowed here
|
2019-03-09 06:03:44 -06:00
|
|
|
LL | x = 1;
|
2023-01-14 21:06:44 -06:00
|
|
|
| ^^^^^ `x` is assigned to here but it was already borrowed
|
2018-08-01 14:40:06 -05:00
|
|
|
LL | f.use_ref();
|
Use larger span for adjustments on method calls
Currently, we use a relatively 'small' span for THIR
expressions generated by an 'adjustment' (e.g. an autoderef,
autoborrow, unsizing). As a result, if a borrow generated
by an adustment ends up causing a borrowcheck error, for example:
```rust
let mut my_var = String::new();
let my_ref = &my_var
my_var.push('a');
my_ref;
```
then the span for the mutable borrow may end up referring
to only the base expression (e.g. `my_var`), rather than
the method call which triggered the mutable borrow
(e.g. `my_var.push('a')`)
Due to a quirk of the MIR borrowck implementation,
this doesn't always get exposed in migration mode,
but it does in many cases.
This commit makes THIR building consistently use 'larger'
spans for adjustment expressions
The intent of this change it make it clearer to users
when it's the specific way in which a variable is
used (for example, in a method call) that produdes
a borrowcheck error. For example, an error message
claiming that a 'mutable borrow occurs here' might
be confusing if it just points at a usage of a variable
(e.g. `my_var`), when no `&mut` is in sight. Pointing
at the entire expression should help to emphasize
that the method call itself is responsible for
the mutable borrow.
In several cases, this makes the `#![feature(nll)]` diagnostic
output match up exactly with the default (migration mode) output.
As a result, several `.nll.stderr` files end up getting removed
entirely.
2021-09-16 15:01:22 -05:00
|
|
|
| ----------- borrow later used here
|
2018-08-01 14:40:06 -05:00
|
|
|
|
|
|
|
error[E0503]: cannot use `x` because it was mutably borrowed
|
2019-04-07 10:07:36 -05:00
|
|
|
--> $DIR/closure-borrow-spans.rs:32:13
|
2018-08-01 14:40:06 -05:00
|
|
|
|
|
|
|
|
LL | let f = || x = 0;
|
|
|
|
| -- - borrow occurs due to use of `x` in closure
|
|
|
|
| |
|
2023-01-14 21:06:44 -06:00
|
|
|
| `x` is borrowed here
|
2019-03-09 06:03:44 -06:00
|
|
|
LL | let y = x;
|
2018-08-01 14:40:06 -05:00
|
|
|
| ^ use of borrowed `x`
|
|
|
|
LL | f.use_ref();
|
Use larger span for adjustments on method calls
Currently, we use a relatively 'small' span for THIR
expressions generated by an 'adjustment' (e.g. an autoderef,
autoborrow, unsizing). As a result, if a borrow generated
by an adustment ends up causing a borrowcheck error, for example:
```rust
let mut my_var = String::new();
let my_ref = &my_var
my_var.push('a');
my_ref;
```
then the span for the mutable borrow may end up referring
to only the base expression (e.g. `my_var`), rather than
the method call which triggered the mutable borrow
(e.g. `my_var.push('a')`)
Due to a quirk of the MIR borrowck implementation,
this doesn't always get exposed in migration mode,
but it does in many cases.
This commit makes THIR building consistently use 'larger'
spans for adjustment expressions
The intent of this change it make it clearer to users
when it's the specific way in which a variable is
used (for example, in a method call) that produdes
a borrowcheck error. For example, an error message
claiming that a 'mutable borrow occurs here' might
be confusing if it just points at a usage of a variable
(e.g. `my_var`), when no `&mut` is in sight. Pointing
at the entire expression should help to emphasize
that the method call itself is responsible for
the mutable borrow.
In several cases, this makes the `#![feature(nll)]` diagnostic
output match up exactly with the default (migration mode) output.
As a result, several `.nll.stderr` files end up getting removed
entirely.
2021-09-16 15:01:22 -05:00
|
|
|
| ----------- borrow later used here
|
2018-08-01 14:40:06 -05:00
|
|
|
|
|
|
|
error[E0502]: cannot borrow `x` as immutable because it is also borrowed as mutable
|
2019-04-07 10:07:36 -05:00
|
|
|
--> $DIR/closure-borrow-spans.rs:38:13
|
2018-08-01 14:40:06 -05:00
|
|
|
|
|
|
|
|
LL | let f = || x = 0;
|
|
|
|
| -- - first borrow occurs due to use of `x` in closure
|
|
|
|
| |
|
|
|
|
| mutable borrow occurs here
|
2019-03-09 06:03:44 -06:00
|
|
|
LL | let y = &x;
|
2018-08-01 14:40:06 -05:00
|
|
|
| ^^ immutable borrow occurs here
|
|
|
|
LL | f.use_ref();
|
Use larger span for adjustments on method calls
Currently, we use a relatively 'small' span for THIR
expressions generated by an 'adjustment' (e.g. an autoderef,
autoborrow, unsizing). As a result, if a borrow generated
by an adustment ends up causing a borrowcheck error, for example:
```rust
let mut my_var = String::new();
let my_ref = &my_var
my_var.push('a');
my_ref;
```
then the span for the mutable borrow may end up referring
to only the base expression (e.g. `my_var`), rather than
the method call which triggered the mutable borrow
(e.g. `my_var.push('a')`)
Due to a quirk of the MIR borrowck implementation,
this doesn't always get exposed in migration mode,
but it does in many cases.
This commit makes THIR building consistently use 'larger'
spans for adjustment expressions
The intent of this change it make it clearer to users
when it's the specific way in which a variable is
used (for example, in a method call) that produdes
a borrowcheck error. For example, an error message
claiming that a 'mutable borrow occurs here' might
be confusing if it just points at a usage of a variable
(e.g. `my_var`), when no `&mut` is in sight. Pointing
at the entire expression should help to emphasize
that the method call itself is responsible for
the mutable borrow.
In several cases, this makes the `#![feature(nll)]` diagnostic
output match up exactly with the default (migration mode) output.
As a result, several `.nll.stderr` files end up getting removed
entirely.
2021-09-16 15:01:22 -05:00
|
|
|
| ----------- mutable borrow later used here
|
2018-08-01 14:40:06 -05:00
|
|
|
|
|
|
|
error[E0499]: cannot borrow `x` as mutable more than once at a time
|
2019-04-07 10:07:36 -05:00
|
|
|
--> $DIR/closure-borrow-spans.rs:44:13
|
2018-08-01 14:40:06 -05:00
|
|
|
|
|
|
|
|
LL | let f = || x = 0;
|
|
|
|
| -- - first borrow occurs due to use of `x` in closure
|
|
|
|
| |
|
|
|
|
| first mutable borrow occurs here
|
2019-03-09 06:03:44 -06:00
|
|
|
LL | let y = &mut x;
|
2018-08-01 14:40:06 -05:00
|
|
|
| ^^^^^^ second mutable borrow occurs here
|
|
|
|
LL | f.use_ref();
|
Use larger span for adjustments on method calls
Currently, we use a relatively 'small' span for THIR
expressions generated by an 'adjustment' (e.g. an autoderef,
autoborrow, unsizing). As a result, if a borrow generated
by an adustment ends up causing a borrowcheck error, for example:
```rust
let mut my_var = String::new();
let my_ref = &my_var
my_var.push('a');
my_ref;
```
then the span for the mutable borrow may end up referring
to only the base expression (e.g. `my_var`), rather than
the method call which triggered the mutable borrow
(e.g. `my_var.push('a')`)
Due to a quirk of the MIR borrowck implementation,
this doesn't always get exposed in migration mode,
but it does in many cases.
This commit makes THIR building consistently use 'larger'
spans for adjustment expressions
The intent of this change it make it clearer to users
when it's the specific way in which a variable is
used (for example, in a method call) that produdes
a borrowcheck error. For example, an error message
claiming that a 'mutable borrow occurs here' might
be confusing if it just points at a usage of a variable
(e.g. `my_var`), when no `&mut` is in sight. Pointing
at the entire expression should help to emphasize
that the method call itself is responsible for
the mutable borrow.
In several cases, this makes the `#![feature(nll)]` diagnostic
output match up exactly with the default (migration mode) output.
As a result, several `.nll.stderr` files end up getting removed
entirely.
2021-09-16 15:01:22 -05:00
|
|
|
| ----------- first borrow later used here
|
2018-08-01 14:40:06 -05:00
|
|
|
|
|
|
|
error[E0597]: `x` does not live long enough
|
2019-04-07 10:07:36 -05:00
|
|
|
--> $DIR/closure-borrow-spans.rs:52:16
|
2018-08-01 14:40:06 -05:00
|
|
|
|
|
2019-03-09 06:03:44 -06:00
|
|
|
LL | f = || x = 0;
|
2018-08-01 14:40:06 -05:00
|
|
|
| -- ^ borrowed value does not live long enough
|
|
|
|
| |
|
|
|
|
| value captured here
|
|
|
|
LL | }
|
|
|
|
| - `x` dropped here while still borrowed
|
|
|
|
LL | f.use_ref();
|
Use larger span for adjustments on method calls
Currently, we use a relatively 'small' span for THIR
expressions generated by an 'adjustment' (e.g. an autoderef,
autoborrow, unsizing). As a result, if a borrow generated
by an adustment ends up causing a borrowcheck error, for example:
```rust
let mut my_var = String::new();
let my_ref = &my_var
my_var.push('a');
my_ref;
```
then the span for the mutable borrow may end up referring
to only the base expression (e.g. `my_var`), rather than
the method call which triggered the mutable borrow
(e.g. `my_var.push('a')`)
Due to a quirk of the MIR borrowck implementation,
this doesn't always get exposed in migration mode,
but it does in many cases.
This commit makes THIR building consistently use 'larger'
spans for adjustment expressions
The intent of this change it make it clearer to users
when it's the specific way in which a variable is
used (for example, in a method call) that produdes
a borrowcheck error. For example, an error message
claiming that a 'mutable borrow occurs here' might
be confusing if it just points at a usage of a variable
(e.g. `my_var`), when no `&mut` is in sight. Pointing
at the entire expression should help to emphasize
that the method call itself is responsible for
the mutable borrow.
In several cases, this makes the `#![feature(nll)]` diagnostic
output match up exactly with the default (migration mode) output.
As a result, several `.nll.stderr` files end up getting removed
entirely.
2021-09-16 15:01:22 -05:00
|
|
|
| ----------- borrow later used here
|
2018-08-01 14:40:06 -05:00
|
|
|
|
|
|
|
error[E0506]: cannot assign to `x` because it is borrowed
|
2019-04-07 10:07:36 -05:00
|
|
|
--> $DIR/closure-borrow-spans.rs:59:5
|
2018-08-01 14:40:06 -05:00
|
|
|
|
|
|
|
|
LL | let f = || x = 0;
|
|
|
|
| -- - borrow occurs due to use in closure
|
|
|
|
| |
|
2023-01-14 21:06:44 -06:00
|
|
|
| `x` is borrowed here
|
2019-03-09 06:03:44 -06:00
|
|
|
LL | x = 1;
|
2023-01-14 21:06:44 -06:00
|
|
|
| ^^^^^ `x` is assigned to here but it was already borrowed
|
2018-08-01 14:40:06 -05:00
|
|
|
LL | f.use_ref();
|
Use larger span for adjustments on method calls
Currently, we use a relatively 'small' span for THIR
expressions generated by an 'adjustment' (e.g. an autoderef,
autoborrow, unsizing). As a result, if a borrow generated
by an adustment ends up causing a borrowcheck error, for example:
```rust
let mut my_var = String::new();
let my_ref = &my_var
my_var.push('a');
my_ref;
```
then the span for the mutable borrow may end up referring
to only the base expression (e.g. `my_var`), rather than
the method call which triggered the mutable borrow
(e.g. `my_var.push('a')`)
Due to a quirk of the MIR borrowck implementation,
this doesn't always get exposed in migration mode,
but it does in many cases.
This commit makes THIR building consistently use 'larger'
spans for adjustment expressions
The intent of this change it make it clearer to users
when it's the specific way in which a variable is
used (for example, in a method call) that produdes
a borrowcheck error. For example, an error message
claiming that a 'mutable borrow occurs here' might
be confusing if it just points at a usage of a variable
(e.g. `my_var`), when no `&mut` is in sight. Pointing
at the entire expression should help to emphasize
that the method call itself is responsible for
the mutable borrow.
In several cases, this makes the `#![feature(nll)]` diagnostic
output match up exactly with the default (migration mode) output.
As a result, several `.nll.stderr` files end up getting removed
entirely.
2021-09-16 15:01:22 -05:00
|
|
|
| ----------- borrow later used here
|
2018-08-01 14:40:06 -05:00
|
|
|
|
|
|
|
error[E0505]: cannot move out of `x` because it is borrowed
|
2019-04-07 10:07:36 -05:00
|
|
|
--> $DIR/closure-borrow-spans.rs:65:13
|
2018-08-01 14:40:06 -05:00
|
|
|
|
|
|
|
|
LL | let f = || *x = 0;
|
2021-03-17 01:51:27 -05:00
|
|
|
| -- -- borrow occurs due to use in closure
|
2018-08-01 14:40:06 -05:00
|
|
|
| |
|
|
|
|
| borrow of `x` occurs here
|
2019-03-09 06:03:44 -06:00
|
|
|
LL | let y = x;
|
2018-08-01 14:40:06 -05:00
|
|
|
| ^ move out of `x` occurs here
|
|
|
|
LL | f.use_ref();
|
Use larger span for adjustments on method calls
Currently, we use a relatively 'small' span for THIR
expressions generated by an 'adjustment' (e.g. an autoderef,
autoborrow, unsizing). As a result, if a borrow generated
by an adustment ends up causing a borrowcheck error, for example:
```rust
let mut my_var = String::new();
let my_ref = &my_var
my_var.push('a');
my_ref;
```
then the span for the mutable borrow may end up referring
to only the base expression (e.g. `my_var`), rather than
the method call which triggered the mutable borrow
(e.g. `my_var.push('a')`)
Due to a quirk of the MIR borrowck implementation,
this doesn't always get exposed in migration mode,
but it does in many cases.
This commit makes THIR building consistently use 'larger'
spans for adjustment expressions
The intent of this change it make it clearer to users
when it's the specific way in which a variable is
used (for example, in a method call) that produdes
a borrowcheck error. For example, an error message
claiming that a 'mutable borrow occurs here' might
be confusing if it just points at a usage of a variable
(e.g. `my_var`), when no `&mut` is in sight. Pointing
at the entire expression should help to emphasize
that the method call itself is responsible for
the mutable borrow.
In several cases, this makes the `#![feature(nll)]` diagnostic
output match up exactly with the default (migration mode) output.
As a result, several `.nll.stderr` files end up getting removed
entirely.
2021-09-16 15:01:22 -05:00
|
|
|
| ----------- borrow later used here
|
2018-08-01 14:40:06 -05:00
|
|
|
|
|
|
|
error[E0501]: cannot borrow `x` as immutable because previous closure requires unique access
|
2019-04-07 10:07:36 -05:00
|
|
|
--> $DIR/closure-borrow-spans.rs:71:13
|
2018-08-01 14:40:06 -05:00
|
|
|
|
|
|
|
|
LL | let f = || *x = 0;
|
2021-03-17 01:51:27 -05:00
|
|
|
| -- -- first borrow occurs due to use of `x` in closure
|
2018-08-01 14:40:06 -05:00
|
|
|
| |
|
|
|
|
| closure construction occurs here
|
2019-03-09 06:03:44 -06:00
|
|
|
LL | let y = &x;
|
2018-11-30 07:55:51 -06:00
|
|
|
| ^^ second borrow occurs here
|
2018-08-01 14:40:06 -05:00
|
|
|
LL | f.use_ref();
|
Use larger span for adjustments on method calls
Currently, we use a relatively 'small' span for THIR
expressions generated by an 'adjustment' (e.g. an autoderef,
autoborrow, unsizing). As a result, if a borrow generated
by an adustment ends up causing a borrowcheck error, for example:
```rust
let mut my_var = String::new();
let my_ref = &my_var
my_var.push('a');
my_ref;
```
then the span for the mutable borrow may end up referring
to only the base expression (e.g. `my_var`), rather than
the method call which triggered the mutable borrow
(e.g. `my_var.push('a')`)
Due to a quirk of the MIR borrowck implementation,
this doesn't always get exposed in migration mode,
but it does in many cases.
This commit makes THIR building consistently use 'larger'
spans for adjustment expressions
The intent of this change it make it clearer to users
when it's the specific way in which a variable is
used (for example, in a method call) that produdes
a borrowcheck error. For example, an error message
claiming that a 'mutable borrow occurs here' might
be confusing if it just points at a usage of a variable
(e.g. `my_var`), when no `&mut` is in sight. Pointing
at the entire expression should help to emphasize
that the method call itself is responsible for
the mutable borrow.
In several cases, this makes the `#![feature(nll)]` diagnostic
output match up exactly with the default (migration mode) output.
As a result, several `.nll.stderr` files end up getting removed
entirely.
2021-09-16 15:01:22 -05:00
|
|
|
| ----------- first borrow later used here
|
2018-08-01 14:40:06 -05:00
|
|
|
|
|
|
|
error[E0501]: cannot borrow `x` as mutable because previous closure requires unique access
|
2019-04-07 10:07:36 -05:00
|
|
|
--> $DIR/closure-borrow-spans.rs:77:13
|
2018-08-01 14:40:06 -05:00
|
|
|
|
|
|
|
|
LL | let f = || *x = 0;
|
2021-03-17 01:51:27 -05:00
|
|
|
| -- -- first borrow occurs due to use of `x` in closure
|
2018-08-01 14:40:06 -05:00
|
|
|
| |
|
|
|
|
| closure construction occurs here
|
2019-03-09 06:03:44 -06:00
|
|
|
LL | let y = &mut x;
|
2018-11-30 07:55:51 -06:00
|
|
|
| ^^^^^^ second borrow occurs here
|
2018-08-01 14:40:06 -05:00
|
|
|
LL | f.use_ref();
|
Use larger span for adjustments on method calls
Currently, we use a relatively 'small' span for THIR
expressions generated by an 'adjustment' (e.g. an autoderef,
autoborrow, unsizing). As a result, if a borrow generated
by an adustment ends up causing a borrowcheck error, for example:
```rust
let mut my_var = String::new();
let my_ref = &my_var
my_var.push('a');
my_ref;
```
then the span for the mutable borrow may end up referring
to only the base expression (e.g. `my_var`), rather than
the method call which triggered the mutable borrow
(e.g. `my_var.push('a')`)
Due to a quirk of the MIR borrowck implementation,
this doesn't always get exposed in migration mode,
but it does in many cases.
This commit makes THIR building consistently use 'larger'
spans for adjustment expressions
The intent of this change it make it clearer to users
when it's the specific way in which a variable is
used (for example, in a method call) that produdes
a borrowcheck error. For example, an error message
claiming that a 'mutable borrow occurs here' might
be confusing if it just points at a usage of a variable
(e.g. `my_var`), when no `&mut` is in sight. Pointing
at the entire expression should help to emphasize
that the method call itself is responsible for
the mutable borrow.
In several cases, this makes the `#![feature(nll)]` diagnostic
output match up exactly with the default (migration mode) output.
As a result, several `.nll.stderr` files end up getting removed
entirely.
2021-09-16 15:01:22 -05:00
|
|
|
| ----------- first borrow later used here
|
2018-08-01 14:40:06 -05:00
|
|
|
|
|
|
|
error[E0597]: `x` does not live long enough
|
2021-03-17 01:51:27 -05:00
|
|
|
--> $DIR/closure-borrow-spans.rs:86:16
|
2018-08-01 14:40:06 -05:00
|
|
|
|
|
2019-03-09 06:03:44 -06:00
|
|
|
LL | f = || *x = 0;
|
2021-03-17 01:51:27 -05:00
|
|
|
| -- ^^ borrowed value does not live long enough
|
2018-08-01 14:40:06 -05:00
|
|
|
| |
|
|
|
|
| value captured here
|
|
|
|
LL | }
|
|
|
|
| - `x` dropped here while still borrowed
|
|
|
|
LL | f.use_ref();
|
Use larger span for adjustments on method calls
Currently, we use a relatively 'small' span for THIR
expressions generated by an 'adjustment' (e.g. an autoderef,
autoborrow, unsizing). As a result, if a borrow generated
by an adustment ends up causing a borrowcheck error, for example:
```rust
let mut my_var = String::new();
let my_ref = &my_var
my_var.push('a');
my_ref;
```
then the span for the mutable borrow may end up referring
to only the base expression (e.g. `my_var`), rather than
the method call which triggered the mutable borrow
(e.g. `my_var.push('a')`)
Due to a quirk of the MIR borrowck implementation,
this doesn't always get exposed in migration mode,
but it does in many cases.
This commit makes THIR building consistently use 'larger'
spans for adjustment expressions
The intent of this change it make it clearer to users
when it's the specific way in which a variable is
used (for example, in a method call) that produdes
a borrowcheck error. For example, an error message
claiming that a 'mutable borrow occurs here' might
be confusing if it just points at a usage of a variable
(e.g. `my_var`), when no `&mut` is in sight. Pointing
at the entire expression should help to emphasize
that the method call itself is responsible for
the mutable borrow.
In several cases, this makes the `#![feature(nll)]` diagnostic
output match up exactly with the default (migration mode) output.
As a result, several `.nll.stderr` files end up getting removed
entirely.
2021-09-16 15:01:22 -05:00
|
|
|
| ----------- borrow later used here
|
2018-08-01 14:40:06 -05:00
|
|
|
|
|
|
|
error[E0506]: cannot assign to `*x` because it is borrowed
|
2019-04-07 10:07:36 -05:00
|
|
|
--> $DIR/closure-borrow-spans.rs:93:5
|
2018-08-01 14:40:06 -05:00
|
|
|
|
|
|
|
|
LL | let f = || *x = 0;
|
2021-03-17 01:51:27 -05:00
|
|
|
| -- -- borrow occurs due to use in closure
|
2018-08-01 14:40:06 -05:00
|
|
|
| |
|
2023-01-14 21:06:44 -06:00
|
|
|
| `*x` is borrowed here
|
2019-03-09 06:03:44 -06:00
|
|
|
LL | *x = 1;
|
2023-01-14 21:06:44 -06:00
|
|
|
| ^^^^^^ `*x` is assigned to here but it was already borrowed
|
2018-08-01 14:40:06 -05:00
|
|
|
LL | f.use_ref();
|
Use larger span for adjustments on method calls
Currently, we use a relatively 'small' span for THIR
expressions generated by an 'adjustment' (e.g. an autoderef,
autoborrow, unsizing). As a result, if a borrow generated
by an adustment ends up causing a borrowcheck error, for example:
```rust
let mut my_var = String::new();
let my_ref = &my_var
my_var.push('a');
my_ref;
```
then the span for the mutable borrow may end up referring
to only the base expression (e.g. `my_var`), rather than
the method call which triggered the mutable borrow
(e.g. `my_var.push('a')`)
Due to a quirk of the MIR borrowck implementation,
this doesn't always get exposed in migration mode,
but it does in many cases.
This commit makes THIR building consistently use 'larger'
spans for adjustment expressions
The intent of this change it make it clearer to users
when it's the specific way in which a variable is
used (for example, in a method call) that produdes
a borrowcheck error. For example, an error message
claiming that a 'mutable borrow occurs here' might
be confusing if it just points at a usage of a variable
(e.g. `my_var`), when no `&mut` is in sight. Pointing
at the entire expression should help to emphasize
that the method call itself is responsible for
the mutable borrow.
In several cases, this makes the `#![feature(nll)]` diagnostic
output match up exactly with the default (migration mode) output.
As a result, several `.nll.stderr` files end up getting removed
entirely.
2021-09-16 15:01:22 -05:00
|
|
|
| ----------- borrow later used here
|
2018-08-01 14:40:06 -05:00
|
|
|
|
|
|
|
error: aborting due to 14 previous errors
|
|
|
|
|
2019-04-17 12:26:38 -05:00
|
|
|
Some errors have detailed explanations: E0499, E0501, E0502, E0503, E0505, E0506, E0597.
|
2018-08-01 14:40:06 -05:00
|
|
|
For more information about an error, try `rustc --explain E0499`.
|