Skip to content

unexpand: rename uflag to utf8#11826

Merged
sylvestre merged 1 commit intouutils:mainfrom
cakebaker:unexpand_rename_uflag
Apr 20, 2026
Merged

unexpand: rename uflag to utf8#11826
sylvestre merged 1 commit intouutils:mainfrom
cakebaker:unexpand_rename_uflag

Conversation

@cakebaker
Copy link
Copy Markdown
Contributor

This PR renames all uses of uflag to utf8. The name uflag is misleading because the variable is the inverse of the flag. I used the name utf8 to match the name we use in expand for the same concept.

@Alonely0
Copy link
Copy Markdown
Contributor

makes sense. grepping the codebase I also got 6 occurrences in id. Wanna go for it as well?

@github-actions
Copy link
Copy Markdown

GNU testsuite comparison:

Skip an intermittent issue tests/date/date-locale-hour (fails in this run but passes in the 'main' branch)
Skip an intermittent issue tests/tail/symlink (fails in this run but passes in the 'main' branch)
Skip an intermittent issue tests/tty/tty-eof (fails in this run but passes in the 'main' branch)
Note: The gnu test tests/cp/link-heap is now being skipped but was previously passing.
Note: The gnu test tests/printf/printf-surprise is now being skipped but was previously passing.
Congrats! The gnu test tests/basenc/bounded-memory is now passing!

@cakebaker
Copy link
Copy Markdown
Contributor Author

I would do it in a different PR, as there the name is not used in a misleading way.

@sylvestre sylvestre merged commit f2f3f7b into uutils:main Apr 20, 2026
168 of 169 checks passed
@cakebaker cakebaker deleted the unexpand_rename_uflag branch April 21, 2026 05:10
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