Skip to content

Fix terminfo::param %i op - #7160

Closed
lilyball wants to merge 1 commit into
rust-lang:masterfrom
lilyball:terminfo-parm-i-fix
Closed

lilyball wants to merge 1 commit into
rust-lang:masterfrom
lilyball:terminfo-parm-i-fix

Conversation

@lilyball

Copy link
Copy Markdown
Contributor

My latest terminfo work introduced a bug in the handling of %i, which was noticed by @huonw after the PR was already merged in.

r? @thestinger

@lilyball lilyball mentioned this pull request Jun 15, 2013
bors added a commit that referenced this pull request Jun 16, 2013
My latest terminfo work introduced a bug in the handling of %i, which was noticed by @huonw after the PR was already merged in.

r? @thestinger
@bors bors closed this Jun 16, 2013
flip1995 pushed a commit to flip1995/rust that referenced this pull request Jun 17, 2021
…dnet,camsteffen

Don't trigger `field_reassign_with_default` in macros

Fixes rust-lang#7155

Producing a good suggestion for this lint is already hard when no macros
are involved. With macros the lint message and the suggestion are just
confusing. Since both, producing a good suggestion and figuring out if
this pattern can be re-written inside a macro is nearly impossible, just
bail out.

changelog: [`field_reassign_with_default`] No longer triggers in macros

---

No that our reviewing queue is under control, I want to start hacking on Clippy myself again. Starting with an easy issue to get back in :)
U007D pushed a commit to U007D/rust-mos that referenced this pull request Aug 21, 2026
7160: Get `hir::Function` return type r=flodiebold a=arnaudgolfouse

Hello !

As said in rust-lang#7158, I noticed that `hir::Function` has no direct way of getting the return type, so this PR adds this functionality.

Co-authored-by: Arnaud <arnaud.golfouse@free.fr>
ytmimi added a commit to ytmimi/rust that referenced this pull request Oct 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants