Skip to content

Commit 32fab73

Browse files
MuscraftMuscraft
committed
feat: Use annotate-snippets by default on nightly
1 parent 018fe95 commit 32fab73

File tree

9 files changed

+102
-68
lines changed

9 files changed

+102
-68
lines changed

‎compiler/rustc_session/src/config.rs‎

Lines changed: 53 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -2050,8 +2050,16 @@ impl JsonUnusedExterns {
20502050
///
20512051
/// The first value returned is how to render JSON diagnostics, and the second
20522052
/// is whether or not artifact notifications are enabled.
2053-
pub fn parse_json(early_dcx: &EarlyDiagCtxt, matches: &getopts::Matches) -> JsonConfig {
2054-
let mut json_rendered = HumanReadableErrorType::Default { short: false };
2053+
pub fn parse_json(
2054+
early_dcx: &EarlyDiagCtxt,
2055+
matches: &getopts::Matches,
2056+
is_nightly_build: bool,
2057+
) -> JsonConfig {
2058+
let mut json_rendered = if is_nightly_build {
2059+
HumanReadableErrorType::AnnotateSnippet { short: false, unicode: false }
2060+
} else {
2061+
HumanReadableErrorType::Default { short: false }
2062+
};
20552063
let mut json_color = ColorConfig::Never;
20562064
let mut json_artifact_notifications = false;
20572065
let mut json_unused_externs = JsonUnusedExterns::No;
@@ -2068,7 +2076,11 @@ pub fn parse_json(early_dcx: &EarlyDiagCtxt, matches: &getopts::Matches) -> Json
20682076
for sub_option in option.split(',') {
20692077
match sub_option {
20702078
"diagnostic-short" => {
2071-
json_rendered = HumanReadableErrorType::Default { short: true }
2079+
json_rendered = if is_nightly_build {
2080+
HumanReadableErrorType::AnnotateSnippet { short: true, unicode: false }
2081+
} else {
2082+
HumanReadableErrorType::Default { short: true }
2083+
};
20722084
}
20732085
"diagnostic-unicode" => {
20742086
json_rendered =
@@ -2102,14 +2114,22 @@ pub fn parse_error_format(
21022114
color_config: ColorConfig,
21032115
json_color: ColorConfig,
21042116
json_rendered: HumanReadableErrorType,
2117+
is_nightly_build: bool,
21052118
) -> ErrorOutputType {
2119+
let default_kind = if is_nightly_build {
2120+
HumanReadableErrorType::AnnotateSnippet { short: false, unicode: false }
2121+
} else {
2122+
HumanReadableErrorType::Default { short: false }
2123+
};
21062124
// We need the `opts_present` check because the driver will send us Matches
21072125
// with only stable options if no unstable options are used. Since error-format
21082126
// is unstable, it will not be present. We have to use `opts_present` not
21092127
// `opt_present` because the latter will panic.
21102128
let error_format = if matches.opts_present(&["error-format".to_owned()]) {
21112129
match matches.opt_str("error-format").as_deref() {
2112-
None | Some("human") => ErrorOutputType::HumanReadable { color_config, .. },
2130+
None | Some("human") => {
2131+
ErrorOutputType::HumanReadable { color_config, kind: default_kind }
2132+
}
21132133
Some("human-annotate-rs") => ErrorOutputType::HumanReadable {
21142134
kind: HumanReadableErrorType::AnnotateSnippet { short: false, unicode: false },
21152135
color_config,
@@ -2121,23 +2141,30 @@ pub fn parse_error_format(
21212141
ErrorOutputType::Json { pretty: true, json_rendered, color_config: json_color }
21222142
}
21232143
Some("short") => ErrorOutputType::HumanReadable {
2124-
kind: HumanReadableErrorType::Default { short: true },
2144+
kind: if is_nightly_build {
2145+
HumanReadableErrorType::AnnotateSnippet { short: true, unicode: false }
2146+
} else {
2147+
HumanReadableErrorType::Default { short: true }
2148+
},
21252149
color_config,
21262150
},
21272151
Some("human-unicode") => ErrorOutputType::HumanReadable {
21282152
kind: HumanReadableErrorType::AnnotateSnippet { short: false, unicode: true },
21292153
color_config,
21302154
},
21312155
Some(arg) => {
2132-
early_dcx.set_error_format(ErrorOutputType::HumanReadable { color_config, .. });
2156+
early_dcx.set_error_format(ErrorOutputType::HumanReadable {
2157+
color_config,
2158+
kind: default_kind,
2159+
});
21332160
early_dcx.early_fatal(format!(
21342161
"argument for `--error-format` must be `human`, `human-annotate-rs`, \
21352162
`human-unicode`, `json`, `pretty-json` or `short` (instead was `{arg}`)"
21362163
))
21372164
}
21382165
}
21392166
} else {
2140-
ErrorOutputType::HumanReadable { color_config, .. }
2167+
ErrorOutputType::HumanReadable { color_config, kind: default_kind }
21412168
};
21422169

21432170
match error_format {
@@ -2185,9 +2212,10 @@ pub fn parse_crate_edition(early_dcx: &EarlyDiagCtxt, matches: &getopts::Matches
21852212
fn check_error_format_stability(
21862213
early_dcx: &EarlyDiagCtxt,
21872214
unstable_opts: &UnstableOptions,
2215+
is_nightly_build: bool,
21882216
format: ErrorOutputType,
21892217
) {
2190-
if unstable_opts.unstable_options {
2218+
if unstable_opts.unstable_options || is_nightly_build {
21912219
return;
21922220
}
21932221
let format = match format {
@@ -2615,16 +2643,25 @@ pub fn build_session_options(early_dcx: &mut EarlyDiagCtxt, matches: &getopts::M
26152643

26162644
let edition = parse_crate_edition(early_dcx, matches);
26172645

2646+
let crate_name = matches.opt_str("crate-name");
2647+
let unstable_features = UnstableFeatures::from_environment(crate_name.as_deref());
26182648
let JsonConfig {
26192649
json_rendered,
26202650
json_color,
26212651
json_artifact_notifications,
26222652
json_timings,
26232653
json_unused_externs,
26242654
json_future_incompat,
2625-
} = parse_json(early_dcx, matches);
2655+
} = parse_json(early_dcx, matches, unstable_features.is_nightly_build());
26262656

2627-
let error_format = parse_error_format(early_dcx, matches, color, json_color, json_rendered);
2657+
let error_format = parse_error_format(
2658+
early_dcx,
2659+
matches,
2660+
color,
2661+
json_color,
2662+
json_rendered,
2663+
unstable_features.is_nightly_build(),
2664+
);
26282665

26292666
early_dcx.set_error_format(error_format);
26302667

@@ -2645,7 +2682,12 @@ pub fn build_session_options(early_dcx: &mut EarlyDiagCtxt, matches: &getopts::M
26452682
early_dcx.early_fatal("--json=timings is unstable and requires using `-Zunstable-options`");
26462683
}
26472684

2648-
check_error_format_stability(early_dcx, &unstable_opts, error_format);
2685+
check_error_format_stability(
2686+
early_dcx,
2687+
&unstable_opts,
2688+
unstable_features.is_nightly_build(),
2689+
error_format,
2690+
);
26492691

26502692
let output_types = parse_output_types(early_dcx, &unstable_opts, matches);
26512693

@@ -2832,8 +2874,6 @@ pub fn build_session_options(early_dcx: &mut EarlyDiagCtxt, matches: &getopts::M
28322874
)
28332875
}
28342876

2835-
let crate_name = matches.opt_str("crate-name");
2836-
let unstable_features = UnstableFeatures::from_environment(crate_name.as_deref());
28372877
// Parse any `-l` flags, which link to native libraries.
28382878
let libs = parse_native_libs(early_dcx, &unstable_opts, unstable_features, matches);
28392879

‎src/librustdoc/config.rs‎

Lines changed: 12 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -391,10 +391,19 @@ impl Options {
391391
}
392392

393393
let color = config::parse_color(early_dcx, matches);
394+
let crate_name = matches.opt_str("crate-name");
395+
let unstable_features =
396+
rustc_feature::UnstableFeatures::from_environment(crate_name.as_deref());
394397
let config::JsonConfig { json_rendered, json_unused_externs, json_color, .. } =
395-
config::parse_json(early_dcx, matches);
396-
let error_format =
397-
config::parse_error_format(early_dcx, matches, color, json_color, json_rendered);
398+
config::parse_json(early_dcx, matches, unstable_features.is_nightly_build());
399+
let error_format = config::parse_error_format(
400+
early_dcx,
401+
matches,
402+
color,
403+
json_color,
404+
json_rendered,
405+
unstable_features.is_nightly_build(),
406+
);
398407
let diagnostic_width = matches.opt_get("diagnostic-width").unwrap_or_default();
399408

400409
let mut target_modifiers = BTreeMap::<OptionsTargetModifiers, String>::new();
@@ -755,7 +764,6 @@ impl Options {
755764
}
756765
};
757766

758-
let crate_name = matches.opt_str("crate-name");
759767
let bin_crate = crate_types.contains(&CrateType::Executable);
760768
let proc_macro_crate = crate_types.contains(&CrateType::ProcMacro);
761769
let playground_url = matches.opt_str("playground-url");
@@ -817,9 +825,6 @@ impl Options {
817825
crate::scrape_examples::load_call_locations(with_examples, dcx, &mut loaded_paths);
818826
let doctest_build_args = matches.opt_strs("doctest-build-arg");
819827

820-
let unstable_features =
821-
rustc_feature::UnstableFeatures::from_environment(crate_name.as_deref());
822-
823828
let disable_minification = matches.opt_present("disable-minification");
824829

825830
let options = Options {

‎src/tools/clippy/tests/ui/bool_assert_comparison.stderr‎

Lines changed: 4 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -272,10 +272,8 @@ LL | assert_eq!(a!(), true);
272272
|
273273
help: replace it with `assert!(..)`
274274
|
275-
LL | true
276-
...
277-
LL |
278-
LL ~ assert!(a!());
275+
LL - assert_eq!(a!(), true);
276+
LL + assert!(a!());
279277
|
280278

281279
error: used `assert_eq!` with a literal bool
@@ -286,10 +284,8 @@ LL | assert_eq!(true, b!());
286284
|
287285
help: replace it with `assert!(..)`
288286
|
289-
LL | true
290-
...
291-
LL |
292-
LL ~ assert!(b!());
287+
LL - assert_eq!(true, b!());
288+
LL + assert!(b!());
293289
|
294290

295291
error: used `debug_assert_eq!` with a literal bool

‎src/tools/clippy/tests/ui/new_without_default.stderr‎

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -191,7 +191,6 @@ LL + fn default() -> Self {
191191
LL + Self::new()
192192
LL + }
193193
LL + }
194-
LL | impl NewWithCfg {
195194
|
196195

197196
error: you should consider adding a `Default` implementation for `NewWith2Cfgs`
@@ -212,7 +211,6 @@ LL + fn default() -> Self {
212211
LL + Self::new()
213212
LL + }
214213
LL + }
215-
LL | impl NewWith2Cfgs {
216214
|
217215

218216
error: you should consider adding a `Default` implementation for `NewWithExtraneous`
@@ -250,7 +248,6 @@ LL + fn default() -> Self {
250248
LL + Self::new()
251249
LL + }
252250
LL + }
253-
LL | impl NewWithCfgAndExtraneous {
254251
|
255252

256253
error: aborting due to 13 previous errors

‎tests/ui/diagnostic-width/non-1-width-unicode-multiline-label.ascii.stderr‎

Lines changed: 15 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,11 @@
11
error[E0369]: cannot add `&str` to `&str`
22
--> $DIR/non-1-width-unicode-multiline-label.rs:8:237
33
|
4-
LL | ...👨👩👧👦👨👩👧👦👨👩👧👦👨👩👧👦👨👩👧👦"; let _a = unicode_is_fun + " really fun!";
5-
| -------------- ^ -------------- &str
6-
| | |
7-
| | `+` cannot be used to concatenate two `&str` strings
8-
| &str
4+
LL | ... 👦👨👩👧👦👨👩👧👦"; let _a = unicode_is_fun + " really fun!";
5+
| -------------- ^ -------------- &str
6+
| | |
7+
| | `+` cannot be used to concatenate two `&str` strings
8+
| &str
99
|
1010
= note: string concatenation requires an owned `String` on the left
1111
help: create an owned `String` from a string reference
@@ -16,11 +16,11 @@ LL | let _ = "👨👩👧👦👨👩👧👦👨👩👧👦👨👩👧
1616
error[E0369]: cannot add `&str` to `&str`
1717
--> $DIR/non-1-width-unicode-multiline-label.rs:10:384
1818
|
19-
LL | ...👧👦👨👩👧👦👨👩👧👦"; let _a = unicode_is_fun + " really fun!";
20-
| -------------- ^ -------------- &str
21-
| | |
22-
| | `+` cannot be used to concatenate two `&str` strings
23-
| &str
19+
LL | ... 👦👨👩👧👦👨👩👧👦"; let _a = unicode_is_fun + " really fun!";
20+
| -------------- ^ -------------- &str
21+
| | |
22+
| | `+` cannot be used to concatenate two `&str` strings
23+
| &str
2424
|
2525
= note: string concatenation requires an owned `String` on the left
2626
help: create an owned `String` from a string reference
@@ -31,11 +31,11 @@ LL | let _ = "👨👩👧👦👨👩👧👦👨👩👧👦👨👩👧
3131
error[E0369]: cannot add `&str` to `&str`
3232
--> $DIR/non-1-width-unicode-multiline-label.rs:12:260
3333
|
34-
LL | ...࿇࿈࿉࿊࿋࿌࿍࿎࿏࿐࿑࿒࿓࿔࿕࿖࿗࿘࿙࿚"; let _a = unicode_is_fun + " really fun!";
35-
| -------------- ^ -------------- &str
36-
| | |
37-
| | `+` cannot be used to concatenate two `&str` strings
38-
| &str
34+
LL | ...࿆࿇࿈࿉࿊࿋࿌࿍࿎࿏࿐࿑࿒࿓࿔࿕࿖࿗࿘࿙࿚"; let _a = unicode_is_fun + " really fun!";
35+
| -------------- ^ -------------- &str
36+
| | |
37+
| | `+` cannot be used to concatenate two `&str` strings
38+
| &str
3939
|
4040
= note: string concatenation requires an owned `String` on the left
4141
help: create an owned `String` from a string reference

‎tests/ui/diagnostic-width/non-whitespace-trimming-unicode.stderr‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,10 @@
11
error[E0308]: mismatched types
22
--> $DIR/non-whitespace-trimming-unicode.rs:5:415
33
|
4-
LL | ...♣♤♥♦♧♨♩♪♫♬♭♮♯♰♱♲♳♴♵♶♷♸♹♺♻♼♽♾♿⚀⚁⚂⚃⚄⚅⚆⚈⚉4"; let _: () = 42; let _: &str = "🦀☀☁☂☃☄★☆☇☈☉☊☋☌☍☎☏☐☑☒☓ ☖☗☘☙☚☛☜☝☞☟☠☡☢☣☤☥☦☧☨☩☪☫☬☭☮☯☰☱☲☳☴☵☶☷☸☹☺☻☼☽ ...
5-
| -- ^^ expected `()`, found integer
6-
| |
7-
| expected due to this
4+
LL | ...♤♥♦♧♨♩♪♫♬♭♮♯♰♱♲♳♴♵♶♷♸♹♺♻♼♽♾♿⚀⚁⚂⚃⚄⚅⚆⚈⚉4"; let _: () = 42; let _: &str = "🦀☀☁☂☃☄★☆☇☈☉☊☋☌☍☎☏☐☑☒☓ ☖☗☘☙☚☛☜☝☞☟☠☡☢☣☤☥☦☧☨☩☪☫☬☭☮☯☰☱☲☳...
5+
| -- ^^ expected `()`, found integer
6+
| |
7+
| expected due to this
88

99
error: aborting due to 1 previous error
1010

‎tests/ui/diagnostic-width/tabs-trimming.stderr‎

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1,20 +1,20 @@
11
error[E0408]: variable `v` is not bound in all patterns
22
--> $DIR/tabs-trimming.rs:9:16
33
|
4-
LL | ... v @ 1 | 2 | 3 => panic!("You gave me too little money {}", v), // Long text here: TTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTT...
5-
| - ^ ^ pattern doesn't bind `v`
6-
| | |
7-
| | pattern doesn't bind `v`
8-
| variable not in all patterns
4+
LL | ... v @ 1 | 2 | 3 => panic!("You gave me too little money {}", v), // Long text here: TTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTT...
5+
| - ^ ^ pattern doesn't bind `v`
6+
| | |
7+
| | pattern doesn't bind `v`
8+
| variable not in all patterns
99

1010
error[E0381]: used binding `v` is possibly-uninitialized
1111
--> $DIR/tabs-trimming.rs:9:67
1212
|
13-
LL | ... v @ 1 | 2 | 3 => panic!("You gave me too little money {}", v), // Long text here: TTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTT...
14-
| - ^ `v` used here but it is possibly-uninitialized
15-
| |
16-
| binding initialized here in some conditions
17-
| binding declared here but left uninitialized
13+
LL | ... v @ 1 | 2 | 3 => panic!("You gave me too little money {}", v), // Long text here: TTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTTT...
14+
| - ^ `v` used here but it is possibly-uninitialized
15+
| |
16+
| binding initialized here in some conditions
17+
| binding declared here but left uninitialized
1818

1919
error: aborting due to 2 previous errors
2020

‎tests/ui/include-macros/mismatched-types.stderr‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,8 @@
11
error[E0308]: mismatched types
22
--> $DIR/file.txt:0:1
33
|
4+
LL |
5+
| ^ expected `&[u8]`, found `&str`
46
|
57
::: $DIR/mismatched-types.rs:2:12
68
|

‎tests/ui/macros/same-sequence-span.stderr‎

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -17,14 +17,8 @@ LL | $(= $z:tt)*
1717
error: `$x:expr` may be followed by `$y:tt`, which is not allowed for `expr` fragments
1818
--> $DIR/same-sequence-span.rs:20:1
1919
|
20-
LL | | // `proc_macro_sequence.rs`.
21-
| |_____________________________^not allowed after `expr` fragments
22-
...
23-
LL | proc_macro_sequence::make_foo!();
24-
| ^-------------------------------
25-
| |
26-
| _in this macro invocation
27-
| |
20+
LL | proc_macro_sequence::make_foo!();
21+
| -------------------------------- in this macro invocation
2822
|
2923
= note: allowed there are: `=>`, `,` or `;`
3024
= note: this error originates in the macro `proc_macro_sequence::make_foo` (in Nightly builds, run with -Z macro-backtrace for more info)

0 commit comments

Comments
 (0)