Skip to content

chore: remove unused Default impl for InflightManager - #28

Closed
detail-app[bot] wants to merge 1 commit into
mainfrom
detail/dead-code/chore-remove-unused-default-impl-for-inflightmanag-e69db8
Closed

detail-app[bot] wants to merge 1 commit into
mainfrom
detail/dead-code/chore-remove-unused-default-impl-for-inflightmanag-e69db8

Conversation

@detail-app

@detail-app detail-app Bot commented Sep 22, 2026

Copy link
Copy Markdown

Remove the never-called impl Default for InflightManager block from
foyer-memory/src/inflight.rs. It has no callers anywhere in the workspace
(InflightManager::new() is used directly instead), is not bound by any
T: Default constraint, and is not part of the crate's public API
(mod inflight is private and InflightManager is not re-exported in
the prelude).

History

Signed-off-by: Detail: Dead Code detail-dead-code@opencode.ai


Dead Code PRs can be configured here.

@detail-app

detail-app Bot commented Oct 22, 2026

Copy link
Copy Markdown
Author

Closing this pull request because it has had no human activity in 30 days (since 2026-09-22) and was opened 30 days ago. Detail closes its pull requests once they are more than 30 days old and have had no human activity for 14 days.

If this change is still useful, reopen the pull request and Detail won't close it again. To keep a Detail pull request open, add the detail-keep-open label to it.

@detail-app detail-app Bot closed this Oct 22, 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.

0 participants