Skip to content

refactor: use slice::get for clarity#11108

Merged
cakebaker merged 1 commit intouutils:mainfrom
xtqqczze:slice-get
Feb 25, 2026
Merged

refactor: use slice::get for clarity#11108
cakebaker merged 1 commit intouutils:mainfrom
xtqqczze:slice-get

Conversation

@xtqqczze
Copy link
Contributor

@xtqqczze xtqqczze commented Feb 25, 2026

use slice::get in a few places for clarity

@sylvestre
Copy link
Contributor

please update the commit message to add "why" you are doing this, thanks :)

@xtqqczze xtqqczze marked this pull request as draft February 25, 2026 13:33
@xtqqczze

This comment was marked as resolved.

@github-actions
Copy link

GNU testsuite comparison:

Skipping an intermittent issue tests/pr/bounded-memory (passes in this run but fails in the 'main' branch)

@xtqqczze xtqqczze changed the title refactor: use slice::get refactor: use slice::get for clarity Feb 25, 2026
@github-actions
Copy link

GNU testsuite comparison:

Skip an intermittent issue tests/tail/inotify-dir-recreate (fails in this run but passes in the 'main' branch)
Skipping an intermittent issue tests/date/date-locale-hour (passes in this run but fails in the 'main' branch)
Congrats! The gnu test tests/cut/bounded-memory is no longer failing!

@xtqqczze xtqqczze marked this pull request as ready for review February 25, 2026 14:55
@cakebaker cakebaker merged commit 071698d into uutils:main Feb 25, 2026
155 of 156 checks passed
@cakebaker
Copy link
Contributor

Thanks!

@xtqqczze
Copy link
Contributor Author

@samueltardieu perhaps there could be a clippy lint for this?

@xtqqczze xtqqczze deleted the slice-get branch February 25, 2026 18:21
@oech3
Copy link
Contributor

oech3 commented Feb 25, 2026

Mergeing this broke #11104 and author clearly knew it...

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.

4 participants