Skip to content

Fix for potential typos#10822

Open
lanceXwq wants to merge 4 commits intouutils:mainfrom
lanceXwq:typofix
Open

Fix for potential typos#10822
lanceXwq wants to merge 4 commits intouutils:mainfrom
lanceXwq:typofix

Conversation

@lanceXwq
Copy link

@lanceXwq lanceXwq commented Feb 8, 2026

Hi! I noticed a potential documentation issue in the od help/manpage:

  • -f is described as “double precision (32-bit)”, aren't 32-bit floats single precision?
  • -i is described as “decimal 8-byte units”; if I’m not mistaken, -i corresponds to C int (typically 4-byte) / -t dI.

This PR updates the wording accordingly. No behavior changes—docs/help only. Thanks!

@github-actions
Copy link

github-actions bot commented Feb 8, 2026

GNU testsuite comparison:

Note: The gnu test tests/tail/tail-n0f is now being skipped but was previously passing.

specified.
od-help-traditional = compatibility mode with one input, offset and label.
od-help-o = octal 2-byte units
od-help-capital-i = decimal 8-byte units
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this isn't correct, help me understand if this is your understanding too:

-i corresponds to 4 bytes but
-I corresponds to 8 bytes

od-help-capital-x = unités hexadécimales 4-octets
od-help-capital-h = unités hexadécimales 4-octets
od-help-e = unités virgule flottante double précision (64-bits)
od-help-f = unités virgule flottante double précision (32-bits)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I only did french for a few years in Canada but I don't remember single being a french term, google translate is saying the term here would be "simple"? I think all of the other reviewers are native french speakers can help me out here.

@ChrisDryden
Copy link
Collaborator

Let me know if you think my comment about the -i vs -I is correct but the -f definitely seems correct to me just might be an issue with the french translation

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