Skip to content

Fix stable clippy string repeat lint#4727

Open
joostjager wants to merge 1 commit into
lightningdevkit:mainfrom
joostjager:fix-lint
Open

Fix stable clippy string repeat lint#4727
joostjager wants to merge 1 commit into
lightningdevkit:mainfrom
joostjager:fix-lint

Conversation

@joostjager

Copy link
Copy Markdown
Contributor

Uncaught by broken CI

Use str::repeat in DebugMsg tests to satisfy current stable clippy.
@ldk-reviews-bot

ldk-reviews-bot commented Jun 19, 2026

Copy link
Copy Markdown

I've assigned @valentinewallace as a reviewer!
I'll wait for their review and will help manage the review process.
Once they submit their review, I'll check if a second reviewer would be helpful.

@ldk-claude-review-bot

Copy link
Copy Markdown
Collaborator

This is a trivial, behavior-preserving change in test code. core::iter::repeat(c).take(n).collect::<String>() is equivalent to "c".repeat(n) for each case, including the multibyte emoji case where "\u{1F600}".repeat(n) produces n emoji characters identical to the iterator version.

No issues found.

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