Skip to content

Conversation

@flip1995
Copy link
Member

@flip1995 flip1995 commented Oct 31, 2025

r? @Manishearth

Cargo.lock update due to Clippy version bump

Jarcho and others added 30 commits September 30, 2025 14:08
…r=GuillaumeGomez

Return spans out of `is_doc_comment` to reduce reliance on `.span()` on attributes

r? `@GuillaumeGomez`
Remove boxes from ast list elements

Less indirection should be better perf.
When matched against `Result` with copyable `Ok` variant, uncopyable scrutinee won't
actually be moved. But `manual_unwrap_or`/`manual_unwrap_or_default`
will force it to move, which can cause problem if scrutinee is used later.

changelog: [`manual_unwrap_or`]: don't lint if not safe to move scrutinee
changelog: [`manual_unwrap_or_default`]: don't lint if not safe to move scrutinee

Signed-off-by: Zihan <zihanli0822@gmail.com>
Update to direct link to the Code of Conduct

changelog: none
The errors were unrelated to the lint...
- make them `verbose` -- the ones spanning multiple lines were
  especially egregious
- give a more descriptive help message
…ust-lang#15911)

Fixes rust-lang/rust-clippy#15910

changelog: [`empty_enum`]: don't lint if all variants happen to be
`cfg`-d out
According to the lint naming guidelines[^1], lint names should use the
plural form.

[^1]: https://rust-lang.github.io/rfcs/0344-conventions-galore.html#lints
@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-clippy Relevant to the Clippy team. labels Oct 31, 2025
@rustbot
Copy link
Collaborator

rustbot commented Oct 31, 2025

⚠️ Warning ⚠️

@Manishearth
Copy link
Member

@bors r+ p=1 roll up=never

@bors
Copy link
Collaborator

bors commented Oct 31, 2025

📌 Commit be5093e has been approved by Manishearth

It is now in the queue for this repository.

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Oct 31, 2025
@Manishearth
Copy link
Member

@bors rollup=never

@rust-log-analyzer
Copy link
Collaborator

The job x86_64-gnu-tools failed! Check out the build log: (web) (plain enhanced) (plain)

Click to see the possible cause of the failure (guessed by this bot)
...........................................        (143/143)
.
======== tests/rustdoc-gui/scrape-examples-layout.goml ========

[ERROR] `tests/rustdoc-gui/scrape-examples-layout.goml` line 2: Error: Navigating frame was detached: for command `go-to: "file://" + |DOC_PATH| + "/scrape_examples/fn.test_many.html"`
`tests/rustdoc-gui/scrape-examples-layout.goml` scrape-examples-layout output:
Protocol error: Connection closed. Most likely the page has been closed.
stack: Error: Protocol error: Connection closed. Most likely the page has been closed.
    at assert (/checkout/obj/build/x86_64-unknown-linux-gnu/test/rustdoc-gui/node_modules/puppeteer-core/lib/cjs/puppeteer/util/assert.js:18:15)
    at CdpPage.close (/checkout/obj/build/x86_64-unknown-linux-gnu/test/rustdoc-gui/node_modules/puppeteer-core/lib/cjs/puppeteer/cdp/Page.js:814:36)
    at async runAllCommands (/checkout/obj/build/x86_64-unknown-linux-gnu/test/rustdoc-gui/node_modules/browser-ui-test/src/index.js:433:13)
    at async innerRunTestCode (/checkout/obj/build/x86_64-unknown-linux-gnu/test/rustdoc-gui/node_modules/browser-ui-test/src/index.js:696:21)
    at async innerRunTests (/checkout/obj/build/x86_64-unknown-linux-gnu/test/rustdoc-gui/node_modules/browser-ui-test/src/index.js:633:17)
    at async runTests (/checkout/obj/build/x86_64-unknown-linux-gnu/test/rustdoc-gui/node_modules/browser-ui-test/src/index.js:824:27)


======== tests/rustdoc-gui/scrape-examples-layout.goml ========

[ERROR] `tests/rustdoc-gui/scrape-examples-layout.goml` line 2: Error: Navigating frame was detached: for command `go-to: "file://" + |DOC_PATH| + "/scrape_examples/fn.test_many.html"`
`tests/rustdoc-gui/scrape-examples-layout.goml` scrape-examples-layout output:
Protocol error: Connection closed. Most likely the page has been closed.
stack: Error: Protocol error: Connection closed. Most likely the page has been closed.
    at assert (/checkout/obj/build/x86_64-unknown-linux-gnu/test/rustdoc-gui/node_modules/puppeteer-core/lib/cjs/puppeteer/util/assert.js:18:15)
    at CdpPage.close (/checkout/obj/build/x86_64-unknown-linux-gnu/test/rustdoc-gui/node_modules/puppeteer-core/lib/cjs/puppeteer/cdp/Page.js:814:36)
    at async runAllCommands (/checkout/obj/build/x86_64-unknown-linux-gnu/test/rustdoc-gui/node_modules/browser-ui-test/src/index.js:433:13)
    at async innerRunTestCode (/checkout/obj/build/x86_64-unknown-linux-gnu/test/rustdoc-gui/node_modules/browser-ui-test/src/index.js:696:21)
    at async innerRunTests (/checkout/obj/build/x86_64-unknown-linux-gnu/test/rustdoc-gui/node_modules/browser-ui-test/src/index.js:633:17)
    at async runTests (/checkout/obj/build/x86_64-unknown-linux-gnu/test/rustdoc-gui/node_modules/browser-ui-test/src/index.js:824:27)


======== tests/rustdoc-gui/search-result-display.goml ========

[WARNING] `tests/rustdoc-gui/search-result-display.goml` line 39: Delta is 0 for "x", maybe try to use `compare-elements-position` instead?

@bors
Copy link
Collaborator

bors commented Nov 1, 2025

⌛ Testing commit be5093e with merge 5eebeb0...

bors added a commit that referenced this pull request Nov 1, 2025
Clippy subtree update

r? `@Manishearth`

Cargo.lock update due to Clippy version bump
@rust-log-analyzer
Copy link
Collaborator

The job dist-aarch64-msvc failed! Check out the build log: (web) (plain enhanced) (plain)

Click to see the possible cause of the failure (guessed by this bot)
[2868/3886] Building CXX object lib\Target\X86\CMakeFiles\LLVMX86CodeGen.dir\X86FastISel.cpp.obj
[2869/3886] Building CXX object lib\Target\X86\CMakeFiles\LLVMX86CodeGen.dir\X86FixupInstTuning.cpp.obj
[2870/3886] Building CXX object lib\Target\X86\CMakeFiles\LLVMX86CodeGen.dir\X86FixupVectorConstants.cpp.obj
[2871/3886] Building CXX object lib\Target\X86\CMakeFiles\LLVMX86CodeGen.dir\X86FixupSetCC.cpp.obj
FAILED: lib/Target/X86/CMakeFiles/LLVMX86CodeGen.dir/X86FixupSetCC.cpp.obj 
sccache C:\a\rust\rust\citools\clang-rust\bin\clang-cl.exe  /nologo -TP -DEXPERIMENTAL_KEY_INSTRUCTIONS -DGTEST_HAS_RTTI=0 -DUNICODE -D_CRT_NONSTDC_NO_DEPRECATE -D_CRT_NONSTDC_NO_WARNINGS -D_CRT_SECURE_NO_DEPRECATE -D_CRT_SECURE_NO_WARNINGS -D_HAS_EXCEPTIONS=0 -D_SCL_SECURE_NO_DEPRECATE -D_SCL_SECURE_NO_WARNINGS -D_UNICODE -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -D__STDC_LIMIT_MACROS -IC:\a\rust\rust\build\aarch64-pc-windows-msvc\llvm\build\lib\Target\X86 -IC:\a\rust\rust\src\llvm-project\llvm\lib\Target\X86 -IC:\a\rust\rust\build\aarch64-pc-windows-msvc\llvm\build\include -IC:\a\rust\rust\src\llvm-project\llvm\include -nologo -MT -Brepro --target=aarch64-pc-windows-msvc /Zc:inline /Zc:__cplusplus /Oi /Brepro /bigobj /permissive- -Werror=unguarded-availability-new /Gw /O2 /Ob2 /DNDEBUG -std:c++17 -MT  /EHs-c- /GR- /showIncludes /Folib\Target\X86\CMakeFiles\LLVMX86CodeGen.dir\X86FixupSetCC.cpp.obj /Fdlib\Target\X86\CMakeFiles\LLVMX86CodeGen.dir\LLVMX86CodeGen.pdb -c -- C:\a\rust\rust\src\llvm-project\llvm\lib\Target\X86\X86FixupSetCC.cpp
sccache: error: An operation on a socket could not be performed because the system lacked sufficient buffer space or because a queue was full. (os error 10055)
[2872/3886] Building CXX object lib\Target\X86\CMakeFiles\LLVMX86CodeGen.dir\X86DynAllocaExpander.cpp.obj
[2873/3886] Building CXX object lib\Target\X86\CMakeFiles\LLVMX86CodeGen.dir\X86AvoidStoreForwardingBlocks.cpp.obj
[2874/3886] Building CXX object lib\Target\X86\CMakeFiles\LLVMX86CodeGen.dir\X86FlagsCopyLowering.cpp.obj
ninja: build stopped: subcommand failed.

@bors
Copy link
Collaborator

bors commented Nov 1, 2025

💔 Test failed - checks-actions

@bors bors added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Nov 1, 2025
@matthiaskrgr
Copy link
Member

@bors retry rollup=iffy

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Nov 1, 2025
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Nov 1, 2025
…Manishearth

Clippy subtree update

r? `@Manishearth`

Cargo.lock update due to Clippy version bump
bors added a commit that referenced this pull request Nov 1, 2025
Rollup of 10 pull requests

Successful merges:

 - #135602 (Tweak output of missing lifetime on associated type)
 - #139751 (Implement pin-project in pattern matching for `&pin mut|const T`)
 - #142682 (Update bundled musl to 1.2.5)
 - #148171 (Simplify code to generate line numbers in highlight)
 - #148263 (Unpin `libc` and `rustix` in `compiler` and `rustbook`)
 - #148301 ([rustdoc search] Include extern crates when filtering on `import`)
 - #148330 (Don't require dlltool with the dummy backend on MinGW)
 - #148338 (cleanup: upstream dropped amx-transpose functionality)
 - #148340 (Clippy subtree update)
 - #148343 (`nonpoison::Condvar` should take `MutexGuard` by reference)

r? `@ghost`
`@rustbot` modify labels: rollup
bors added a commit that referenced this pull request Nov 1, 2025
Rollup of 10 pull requests

Successful merges:

 - #135602 (Tweak output of missing lifetime on associated type)
 - #139751 (Implement pin-project in pattern matching for `&pin mut|const T`)
 - #142682 (Update bundled musl to 1.2.5)
 - #148171 (Simplify code to generate line numbers in highlight)
 - #148263 (Unpin `libc` and `rustix` in `compiler` and `rustbook`)
 - #148301 ([rustdoc search] Include extern crates when filtering on `import`)
 - #148330 (Don't require dlltool with the dummy backend on MinGW)
 - #148338 (cleanup: upstream dropped amx-transpose functionality)
 - #148340 (Clippy subtree update)
 - #148343 (`nonpoison::Condvar` should take `MutexGuard` by reference)

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 1ca21ed into rust-lang:master Nov 1, 2025
9 of 12 checks passed
@rustbot rustbot added this to the 1.93.0 milestone Nov 1, 2025
rust-timer added a commit that referenced this pull request Nov 1, 2025
Rollup merge of #148340 - flip1995:clippy-subtree-update, r=Manishearth

Clippy subtree update

r? ``@Manishearth``

Cargo.lock update due to Clippy version bump
@bors
Copy link
Collaborator

bors commented Nov 1, 2025

⌛ Testing commit be5093e with merge fca2e94...

@flip1995 flip1995 deleted the clippy-subtree-update branch November 3, 2025 11:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-clippy Relevant to the Clippy team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.