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

New Issue

claunia · 2026-01-29T17:22:58Z

claunia commented

2026-01-29 17:22:58 +00:00

📋 Pull Request Information

Original PR: https://github.com/CCExtractor/ccextractor/pull/1834
Author: @gaurav02081
Created: 12/15/2025
Status: ✅ Merged
Merged: 12/16/2025
Merged by: @cfsmp3

Base: master ← Head: gaurav-v1

📝 Commits (1)

6ebf98e Fix typos in encoder warning and comment

📊 Changes

1 file changed (+2 additions, -2 deletions)

View changed files

📝 src/lib_ccx/ccx_encoders_common.c (+2 -2)

📄 Description

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.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/CCExtractor/ccextractor/pull/1834 **Author:** [@gaurav02081](https://github.com/gaurav02081) **Created:** 12/15/2025 **Status:** ✅ Merged **Merged:** 12/16/2025 **Merged by:** [@cfsmp3](https://github.com/cfsmp3) **Base:** `master` ← **Head:** `gaurav-v1` --- ### 📝 Commits (1) - [`6ebf98e`](https://github.com/CCExtractor/ccextractor/commit/6ebf98ea4a2b5323ff6c52a3ce99090514c83f54) Fix typos in encoder warning and comment ### 📊 Changes **1 file changed** (+2 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `src/lib_ccx/ccx_encoders_common.c` (+2 -2) </details> ### 📄 Description 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. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

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#2589