[BUG] Capitalization doesn't work with tags #544

Open
opened 2026-01-29 16:47:09 +00:00 by claunia · 0 comments
Owner

Originally created by @NilsIrl on GitHub (Jan 23, 2020).

CCExtractor version: 1764aa1f92

In raising this issue, I confirm the following (please check boxes, eg [X] - and delete unchecked ones):

  • I have read and understood the contributors guide.
  • I have checked that the bug-fix I am reporting can be replicated, or that the feature I am suggesting isn't already present.
  • I have checked that the issue I'm posting isn't already reported.
  • I have checked that the issue I'm porting isn't already solved and no duplicates exist in closed issues and in opened issues
  • I have checked the pull requests tab for existing solutions/implementations to my issue/suggestion.
  • I have used the latest available version of CCExtractor to verify this issue exists.

My familiarity with the project is as follows:

  • I am an active contributor to CCExtractor.

Necessary information

  • Is this a regression (did it work before)? [ ] NO | [ ] YES - I don't know though it probably isn't
  • What platform did you use? [ ] Windows - [X] Linux - [ ] Mac
  • What were the used arguments? ./ccextractor -sc files/CNN.ts

Video links (replace text below with your links)

Additional information

Notice how the word "take" isn't capitalized correctly (It should be "Take").

54c54
<   <i> Take a closer look at GEICO.</i>
---
>   <i> take a closer look at geico.</i>

(This snippet is from line 54.)

Originally created by @NilsIrl on GitHub (Jan 23, 2020). CCExtractor version: 1764aa1f92a2c3c201d7f37b67ee18b551b982ef **In raising this issue, I confirm the following (please check boxes, eg [X] - and delete unchecked ones):** - [X] I have read and understood the [contributors guide](https://github.com/CCExtractor/ccextractor/blob/master/.github/CONTRIBUTING.md). - [X] I have checked that the bug-fix I am reporting can be replicated, or that the feature I am suggesting isn't already present. - [X] I have checked that the issue I'm posting isn't already reported. - [X] I have checked that the issue I'm porting isn't already solved and no duplicates exist in [closed issues](https://github.com/CCExtractor/ccextractor/issues?q=is%3Aissue+is%3Aclosed) and in [opened issues](https://github.com/CCExtractor/ccextractor/issues) - [X] I have checked the pull requests tab for existing solutions/implementations to my issue/suggestion. - [X] I have used the latest available version of CCExtractor to verify this issue exists. **My familiarity with the project is as follows:** - [X] I am an active contributor to CCExtractor. **Necessary information** - Is this a regression (did it work before)? [ ] NO | [ ] YES - `I don't know though it probably isn't` - What platform did you use? [ ] Windows - [X] Linux - [ ] Mac - What were the used arguments? `./ccextractor -sc files/CNN.ts` **Video links (replace text below with your links)** * `CNN.ts` here: https://drive.google.com/drive/folders/0B_61ywKPmI0Ta2diT3l0eTlHc2c **Additional information** Notice how the word "take" isn't capitalized correctly (It should be "Take"). ```patch 54c54 < <i> Take a closer look at GEICO.</i> --- > <i> take a closer look at geico.</i> ``` (This snippet is from line 54.)
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/ccextractor#544