| Age | Commit message (Collapse) | Author | Lines |
|
|
|
|
|
UI Test Cleanup: Split up checked_unwrap tests
Let's slowly bring that ticket closer to the finish line :snail: :checkered_flag:
This splits up `tests/ui/checked_unwrap.rs` into:
* `tests/ui/checked_unwrap/complex.rs`
* `tests/ui/checked_unwrap/simple.rs`
Based on the naming of the methods in the tests.
changelog: none
cc #2038
|
|
for cleaner stderr file
Co-Authored-By: Philipp Krones <hello@philkrones.com>
|
|
Co-Authored-By: Philipp Krones <hello@philkrones.com>
|
|
|
|
This splits up `tests/ui/checked_unwrap.rs` into:
* `tests/ui/checked_unwrap/complex.rs`
* `tests/ui/checked_unwrap/simple.rs`
Based on the naming of the methods in the tests.
cc #2038
|
|
Abstracted repeating strings into statics.
|
|
Fix float_cmp false positive when comparing signum
fixes #4248
changelog: Fix float_cmp false positive when comparing signum
|
|
This moves the `out_of_bounds_indexing` lint tests to their own
directory.
|
|
Check for changelog entry in PR bodies
cc #4031 but now on the auto and try branches. (https://github.com/rust-lang/rust-clippy/issues/3955#issuecomment-502230021)
changelog: none
|
|
|
|
f1.signum() == f2.signum()
f1.signum() != f2.signum()
should not trigger a warning.
|
|
|
|
|
|
|
|
|
|
- Handle returning macro statements properly
- Handle functions that return never type
- Handle functions that panic but do not return never type
|
|
Deny warnings in CI
changelog: none
|
|
|
|
Fix Travis Windows build
Closes #3306
|
|
|
|
This reverts commit 876a7e1f01cb14a63eee54cb44524a87161d5454.
Using incremental build on windows increases the build time on travis by
about 8 minutes.
|
|
|
|
|
|
|
|
|
|
|
|
Maybe uname == Linux was true on the windows VM?
This could be a way to avoid the secret environment variable issue with
Travis CI.
|
|
|
|
don't strip blank lines in lint documentation
changelog: don't strip blank lines in lint documentation
Fixes #4116.
This PR also switches the docs headings to deterministically display in the order that they are declared in the source, with "Configuration" always appearing last. It doesn't seem like there was a defined order before.
|
|
Add dev fmt subcommand
changelog: none
|
|
Fix bug in `implicit_hasher` causing crashes
Skip linting if the type is from an external macro. Closes #4260.
changelog: Fix bug in `implicit_hasher` causing crashes
|
|
|
|
Rustup `macro expansion and resolution`
Rustup https://github.com/rust-lang/rust/pull/62476
changelog: none
|
|
|
|
Skip linting if the type is from an external macro. Closes #4260.
|
|
|
|
|
|
Improve cast_ptr_alignment lint
<!--
Thank you for making Clippy better!
We're collecting our changelog from pull request descriptions.
If your PR only updates to the latest nightly, you can leave the
`changelog` entry as `none`. Otherwise, please write a short comment
explaining your change.
If your PR fixes an issue, you can add "fixes #issue_number" into this
PR description. This way the issue will be automatically closed when
your PR is merged.
If you added a new lint, here's a checklist for things that will be
checked during review or continuous integration.
- [x] Followed [lint naming conventions][lint_naming]
- [x] Added passing UI tests (including committed `.stderr` file)
- [x] `cargo test` passes locally
- [x] Executed `util/dev update_lints`
- [x] Added lint documentation
- [x] Run `cargo fmt`
Note that you can skip the above if you are just opening a WIP PR in
order to get feedback.
Delete this line and everything above before opening your PR -->
* print alignment in bytes in the lint message
* ignore ZST left-hand types
Fixes #3797 and #4256
changelog:
* `cast_ptr_alignment`: Print alignment in bytes in the lint message
* `cast_ptr_alignment`: Ignore casting from ZST left-hand types
|
|
* print alignment in bytes in the lint message
* ignore ZST left-hand types
|
|
cast_ptr_alignment: Mention legal use under known problems
Refs #2881.
changelog: Mention know problems for cast_ptr_alignment
|
|
Refs #2881.
|
|
Disable AppVeyor builds on the master branch
AppVeyor is already checked on every merge of a PR, rechecking it
immediately after on the master branch is not necessary.
Resolves #4263
changelog: none
|
|
AppVeyor is already checked on every merge of a PR, rechecking it
immediately after on the master branch is not necessary.
|
|
Avoid reporting string_lit_as_bytes for long strings
Port of @jens1o code ([b76f939][jens1o_commit])
Fixes #1208
[jens1o_commit]: https://github.com/jens1o/rust-clippy/commit/b76f939ac2efcfe24900c286b3b7713d972d9088
<!--
Thank you for making Clippy better!
We're collecting our changelog from pull request descriptions.
If your PR only updates to the latest nightly, you can leave the
`changelog` entry as `none`. Otherwise, please write a short comment
explaining your change.
If your PR fixes an issue, you can add "fixes #issue_number" into this
PR description. This way the issue will be automatically closed when
your PR is merged.
If you added a new lint, here's a checklist for things that will be
checked during review or continuous integration.
- [ ] Followed [lint naming conventions][lint_naming]
- [ ] Added passing UI tests (including committed `.stderr` file)
- [ ] `cargo test` passes locally
- [ ] Executed `util/dev update_lints`
- [ ] Added lint documentation
- [ ] Run `cargo fmt`
Note that you can skip the above if you are just opening a WIP PR in
order to get feedback.
Delete this line and everything above before opening your PR -->
changelog: bugfix for long strings as bytes
|
|
Use empty block instead of unit type for needless return
fixes #4238
changelog: Use empty block instead of unit type for needless return
|
|
|
|
|
|
Port of @jens1o code ([b76f939][jens1o_commit])
Fixes #1208
[jens1o_commit]: https://github.com/jens1o/rust-clippy/commit/b76f939ac2efcfe24900c286b3b7713d972d9088
Co-authored-by: Thiago Arrais <thiago.arrais@gmail.com>
|