[PR #1834] [FIX] Correct typos in warning message and code comment #2594

Closed
opened 2026-01-29 17:22:58 +00:00 by claunia · 0 comments
Owner

Original Pull Request: https://github.com/CCExtractor/ccextractor/pull/1834

State: closed
Merged: Yes


This PR fixes two small typos in src/lib_ccx/ccx_encoders_common.c.

What was changed

  • Fixed a typo in a user-facing warning message:
    • "Unable tp write UTF BOM""Unable to write UTF BOM"
  • Corrected a typo in an internal TODO comment:
    • "carry about""care about"

Why this change

  • Improves clarity and professionalism of user-facing log output.
  • Improves readability of internal comments for future contributors.
  • No functional or behavioral changes.

Testing

  • No functional testing required (comment and log text only).
  • Build was already verified locally before this change.

In raising this pull request, I confirm the following:

  • I have read and understood the contributors guide.
  • I have checked that another pull request for this purpose does not exist.
  • I have considered, and confirmed that this submission will be valuable to others.
  • I accept that this submission may not be used, and the pull request closed at the will of the maintainer.
  • I give this submission freely, and claim no ownership to its content.

My familiarity with the project is as follows:

  • I have used CCExtractor just a couple of times.
**Original Pull Request:** https://github.com/CCExtractor/ccextractor/pull/1834 **State:** closed **Merged:** Yes --- This PR fixes two small typos in `src/lib_ccx/ccx_encoders_common.c`. ### What was changed - Fixed a typo in a user-facing warning message: - `"Unable tp write UTF BOM"` → `"Unable to write UTF BOM"` - Corrected a typo in an internal TODO comment: - `"carry about"` → `"care about"` ### Why this change - Improves clarity and professionalism of user-facing log output. - Improves readability of internal comments for future contributors. - No functional or behavioral changes. ### Testing - No functional testing required (comment and log text only). - Build was already verified locally before this change. **In raising this pull request, I confirm the following:** - [x] I have read and understood the contributors guide. - [x] I have checked that another pull request for this purpose does not exist. - [x] I have considered, and confirmed that this submission will be valuable to others. - [x] I accept that this submission may not be used, and the pull request closed at the will of the maintainer. - [x] I give this submission freely, and claim no ownership to its content. **My familiarity with the project is as follows:** - [x] I have used CCExtractor just a couple of times.
claunia added the pull-request label 2026-01-29 17:22:58 +00:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/ccextractor#2594