Skip to content

Conversation

@Moredread
Copy link
Contributor

Summary of changes

This pull request includes following changes or fixes.

  • Fix spelling and grammar errors across 9 message files
  • Fix format specifier bug in tape_linux_sg messages

Description

Reviewed all messages/*/root.txt files for spelling and grammar issues. Found and fixed various typos, grammar errors, and one actual bug.

Spelling/Grammar fixes include:

  • "is failed" → "has failed" (verb tense)
  • "failed to determined" → "failed to determine" (infinitive form)
  • "Enexpected" → "Unexpected" (typo)
  • "version os LTFS" → "version of LTFS" (typo)
  • "access to directory" → "access directory" (incorrect preposition)
  • "look an index" → "look like an index" (missing word)
  • "Reached to EOD" → "Reached EOD" (incorrect preposition)
  • "The opcode" → "the opcode" (capitalization after comma)
  • "Error" → "error" (mid-sentence capitalization)
  • Various double periods and missing punctuation

Bug fix:

  • Format specifier 0x02%x0x%02x in messages 30203I and 30244I
  • This caused incorrect output like "M = 0x0234" instead of "M = 0x34"

Type of change

  • Bug fix (non-breaking change which fixes an issue)

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have confirmed my fix is effective or that my feature works

This PR was created with assistance from Claude Code.

Moredread and others added 2 commits December 10, 2025 11:31
internal_error/root.txt:
- I1128E: "is failed" -> "has failed"
- I5004E: "a tags is" -> "if a tag is"
- I5007E: "Do not find" -> "Could not find"

libltfs/root.txt:
- 11193E: "failed to determined" -> "failed to determine"
- 11200E: "failed to determined" -> "failed to determine"
- 11343I: Remove double period at end
- 17073I: Remove period after colon
- Line 533: Remove stray "v" character
- Line 543: "cropto" -> "crypto" in comment

bin_ltfs/root.txt:
- 14120W: "access to directory" -> "access directory"
- 14448I: Add missing closing parenthesis

bin_ltfsck/root.txt:
- 16062I: "Roll back" -> "Rolling back"
- 16101E: "version os LTFS" -> "version of LTFS"
- 16113W: "access to directory" -> "access directory"

bin_ltfsindextool/root.txt:
- 19530I: "look an index" -> "look like an index"
- 19534I: "Reached to EOD" -> "Reached EOD"
- 19535I: "Reached to EOD" -> "Reached EOD"

tape_freebsd_cam/root.txt:
- 31239E: "Error" -> "error", add period
- 31247E: "Error" -> "error", add period

tape_iokit/root.txt:
- 30800-30803I: "The opcode" -> "the opcode"

tape_linux_lin_tape/root.txt:
- 30443W: "Enexpected" -> "Unexpected"
- 30444W: "is failed" -> "has failed"

tape_linux_sg/root.txt:
- 30261W: "failed to write to communicate" -> "failed to communicate"
- 30281W: "is failed" -> "has failed"
- 30283W: "is failed" -> "has failed"

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
The format "0x02%x" prints literal "02" followed by hex value,
instead of the intended zero-padded 2-digit hex "0x%02x".

This caused incorrect output like "M = 0x0234" instead of "M = 0x34".

- 30203I: "0x02%x" -> "0x%02x"
- 30244I: "0x02%x" -> "0x%02x"

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Copy link
Member

@piste-jp piste-jp left a comment

Choose a reason for hiding this comment

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

Thank you for your contribution.

Looks good!

@piste-jp piste-jp merged commit eb48b79 into LinearTapeFileSystem:master Dec 11, 2025
8 checks passed
@Moredread Moredread deleted the spelling branch December 13, 2025 23:27
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