diff options
| author | bors <bors@rust-lang.org> | 2022-05-18 17:15:38 +0000 |
|---|---|---|
| committer | bors <bors@rust-lang.org> | 2022-05-18 17:15:38 +0000 |
| commit | b6ad6fc5aa0fa4c4958b009f5625b44bce674c4f (patch) | |
| tree | 4b0e43f3f01cbbd63d442db514568b1fbd021fb3 /compiler/rustc_codegen_llvm/src | |
| parent | bf2e63104ddb2e596f74b21502a71f9540f11d2e (diff) | |
| parent | 564725775bd200de131b96f4d93a36bd64508e72 (diff) | |
| download | rust-b6ad6fc5aa0fa4c4958b009f5625b44bce674c4f.tar.gz rust-b6ad6fc5aa0fa4c4958b009f5625b44bce674c4f.zip | |
Auto merge of #8823 - smoelius:unknown-field, r=xFrednet
Improve "unknown field" error messages
Fixes #8806
Sample output:
```
error: error reading Clippy's configuration file `/home/smoelius/github/smoelius/rust-clippy/clippy.toml`: unknown field `foobar`, expected one of
allow-expect-in-tests enable-raw-pointer-heuristic-for-send standard-macro-braces
allow-unwrap-in-tests enforced-import-renames third-party
allowed-scripts enum-variant-name-threshold too-large-for-stack
array-size-threshold enum-variant-size-threshold too-many-arguments-threshold
avoid-breaking-exported-api literal-representation-threshold too-many-lines-threshold
await-holding-invalid-types max-fn-params-bools trivial-copy-size-limit
blacklisted-names max-include-file-size type-complexity-threshold
cargo-ignore-publish max-struct-bools unreadable-literal-lint-fractions
cognitive-complexity-threshold max-suggested-slice-pattern-length upper-case-acronyms-aggressive
cyclomatic-complexity-threshold max-trait-bounds vec-box-size-threshold
disallowed-methods msrv verbose-bit-mask-threshold
disallowed-types pass-by-value-size-limit warn-on-all-wildcard-imports
doc-valid-idents single-char-binding-names-threshold
at line 1 column 1
```
You can test this by (say) adding `foobar = 42` to Clippy's root `clippy.toml` file, and running `cargo run --bin cargo-clippy`.
Note that, to get the terminal width, this PR adds `termize` as a dependency to `cargo-clippy`. However, `termize` is also [how `rustc_errors` gets the terminal width](https://github.com/rust-lang/rust/blob/481db40311cdd241ae4d33f34f2f75732e44d8e8/compiler/rustc_errors/src/emitter.rs#L1607). So, hopefully, this is not a dealbreaker.
r? `@xFrednet`
changelog: Enhancements: the "unknown field" error messages for config files now wraps the field names.
Diffstat (limited to 'compiler/rustc_codegen_llvm/src')
0 files changed, 0 insertions, 0 deletions
