Skip to content

avoid unnecessary clone - #14250

Draft
xtqqczze wants to merge 4 commits into
uutils:mainfrom
xtqqczze:rm-clone
Draft

avoid unnecessary clone#14250
xtqqczze wants to merge 4 commits into
uutils:mainfrom
xtqqczze:rm-clone

Conversation

@xtqqczze

@xtqqczze xtqqczze commented Aug 29, 2026

Copy link
Copy Markdown
Contributor

Avoid unnecessary clone() calls in several code paths to reduce redundant allocations and improve efficiency.

@github-actions

github-actions Bot commented Aug 29, 2026

Copy link
Copy Markdown

Binary size comparison:

Individual binary size comparison VS main (threshold: >=5% AND >=4 KB).

Total size of compared binaries: 152.18 MB (+1000 KB, +0.65%)

Significant per-binary changes:
  comm     1.12 MB ->    2.33 MB  (+1.21 MB, +107.29%)

@github-actions

github-actions Bot commented Aug 29, 2026

Copy link
Copy Markdown

GNU testsuite comparison:

Skip an intermittent issue tests/cut/bounded-memory (fails in this run but passes in the 'main' branch)
Skip an intermittent issue tests/date/date-locale-hour (fails in this run but passes in the 'main' branch)
Skipping an intermittent issue tests/rm/isatty (passes in this run but fails in the 'main' branch)

@xtqqczze
xtqqczze marked this pull request as ready for review August 30, 2026 01:34
@xtqqczze
xtqqczze marked this pull request as draft August 30, 2026 10:04
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.

2 participants