Skip to content

Conversation

km274
Copy link

@km274 km274 commented Sep 10, 2025

Fixes #5955

I understand that there's a feature freeze right now. I had started working on this before I was aware of the feature freeze. I don't mind waiting as long as I need to for feedback.

changelog: [same_length_and_capacity]: adds a new lint that checks for usages of from_raw_parts where the same expression is passed for the length and the capacity

Summary Notes

Managed by @rustbot—see help for details

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Sep 10, 2025
@rustbot
Copy link
Collaborator

rustbot commented Sep 10, 2025

r? @llogiq

rustbot has assigned @llogiq.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

Copy link

github-actions bot commented Sep 10, 2025

Lintcheck changes for d6a455e

Lint Added Removed Changed
clippy::same_length_and_capacity 71 0 0

This comment will be updated if you push new changes

@rustbot rustbot added S-blocked Status: marked as blocked ❌ on something else such as an RFC or other implementation work A-lint Area: New lints and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties labels Sep 10, 2025
@km274
Copy link
Author

km274 commented Sep 10, 2025

Oops, I understand why CI failed. I'll take care of that.

@samueltardieu samueltardieu added S-waiting-on-author Status: This is awaiting some action from the author. (Use `@rustbot ready` to update this status) and removed S-blocked Status: marked as blocked ❌ on something else such as an RFC or other implementation work labels Sep 17, 2025
@km274 km274 force-pushed the from_raw_parts_same_len_and_cap branch from 10aca0a to 2a4a572 Compare October 3, 2025 21:29
@rustbot

This comment has been minimized.

@km274
Copy link
Author

km274 commented Oct 3, 2025

@rustbot ready

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties and removed S-waiting-on-author Status: This is awaiting some action from the author. (Use `@rustbot ready` to update this status) labels Oct 3, 2025
@km274 km274 force-pushed the from_raw_parts_same_len_and_cap branch 3 times, most recently from de87dd6 to b3cbb63 Compare October 3, 2025 21:46
@rustbot

This comment has been minimized.

@km274 km274 force-pushed the from_raw_parts_same_len_and_cap branch from b3cbb63 to d6a455e Compare October 7, 2025 01:34
@rustbot
Copy link
Collaborator

rustbot commented Oct 7, 2025

This PR was rebased onto a different master commit. Here's a range-diff highlighting what actually changed.

Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-lint Area: New lints S-waiting-on-review Status: Awaiting review from the assignee but also interested parties
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Lint for using the same expression for both length and capacity when constructing a vec/string
4 participants