]> git.lizzy.rs Git - rust.git/commit
Rollup merge of #47407 - gaurikholkar:master, r=estebank
authorkennytm <kennytm@gmail.com>
Wed, 17 Jan 2018 17:57:16 +0000 (01:57 +0800)
committerGitHub <noreply@github.com>
Wed, 17 Jan 2018 17:57:16 +0000 (01:57 +0800)
commitbd8aa02ff4b76a6aa9c03ad8ab80aaab21080e9c
treeb2988ced186bb5dd1c5bf5118c2b6e7b959964fe
parente7087f0f4f5300734f37498d4ed9059aafdfcf0a
parentefe3d69ad827cc359b1a120b0e5c58dd87a2935b
Rollup merge of #47407 - gaurikholkar:master, r=estebank

fix mispositioned span

This fixes #47377

The output now looks like this
```
error[E0369]: binary operation `+` cannot be applied to type `&str`
 --> h.rs:3:11
  |
3 |     let _a = b + ", World!";
  |              ^^^^^^^^^^^^^^ `+` can't be used to concatenate two `&str` strings
help: `to_owned()` can be used to create an owned `String` from a string reference. String concatenation appends the string on the right to the string on the left and may require reallocation. This requires ownership of the string on the left
  |
3 |     let _a = b.to_owned() + ", World!";
  |              ^^^^^^^^^

error: aborting due to previous error
```
For the case when emojis are involved,  it gives the new output for proper indentation.
But for an indentation as follows,
```
fn main() {
let b = "hello";
    let _a = b + ", World!";
}
```
it still mispositions the span
```
3 |     println!("🦀🦀🦀🦀🦀"); let _a = b + ", World!";
  |                                           ^^^^^^^^^^^^^^ `+` can't be used to concatenate two `&str` strings
  |
3 |     println!("🦀🦀🦀🦀🦀"); let _a = b.to_owned() + ", World!";
  |                                           ^^^^^^^
error: aborting due to previous erro
```

cc @estebank  @est31
src/librustc_errors/emitter.rs