Uh oh!
There was an error while loading. Please reload this page.
Logging fixes - #8250
Closed
brson wants to merge 2 commits into
Closed
Conversation
…t formatting tests to be compatible with both the new and old runtimes
Contributor
There was a problem hiding this comment.
Doesn't this make a string of length 257 into one of length 261? (expected behaviour?)
ContributorAuthor
There was a problem hiding this comment.
Looks like it. I don't think it's a problem though. The goal is only to avoid spewing endless garbage - it doesn't need to be precise.
ContributorAuthor
There was a problem hiding this comment.
On further consideration though, this is probably only appropriate behavior for the console logger, so likely belongs somewhere else. In theory logging can go to arbitrary sinks. Can be fixed in the future. Logging is going to be completely overhauled eventually anyway.
thestinger
commented
Aug 3, 2013
Contributor
Actually landing with #8257. |
flip1995 pushed a commit
to flip1995/rust
that referenced
this pull request
Feb 10, 2022
Fix underflow in `manual_split_once` lint
Hi, a friend found clippy started crashing on a suspiciously large allocation of `u64::MAX` memory on their code.
The mostly minimized repro is:
```rust
fn _f01(title: &str) -> Option<()> {
let _ = title[1..].splitn(2, '[').next()?;
Some(())
}
```
The underflow happens in this case on line 57 of the patch but I've changed the other substraction to saturating as well since it could potentially cause the same issue.
I'm not sure where to put a regression test, or if it's even worth for such a thing.
Aside, has it been considered before to build clippy with overflow checks enabled?
changelog: fix ICE of underflow in `manual_split_once` lintU007D pushed a commit
to U007D/rust-mos
that referenced
this pull request
Aug 21, 2026
8250: More accurately classify assoc. types in paths r=jonas-schievink a=jonas-schievink Previously `Iterator<Whoops$0 = ()>` would go to the `Iterator` trait. This fixes that and correctly marks `Whoops` as unresolved. bors r+ Co-authored-by: Jonas Schievink <jonasschievink@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Updated from #8241