[PR #1139] [MERGED] [IMPROVEMENT] Filter bad words #1915

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

📋 Pull Request Information

Original PR: https://github.com/CCExtractor/ccextractor/pull/1139
Author: @NilsIrl
Created: 12/8/2019
Status: Merged
Merged: 1/5/2020
Merged by: @canihavesomecoffee

Base: masterHead: filter_bad_words


📝 Commits (10+)

  • 5b29db3 Remove space before ';'
  • 99a12b8 Add --kf option and parse files
  • 7d8499a Rename profanity_file to filter_profanity_file. Dump params
  • a7d2264 Use correct function
  • 2739602 Add missing continue
  • 8ef89f6 Fix double free error
  • 59a8c7a Censor word when in dictionary
  • 37e4d41 Fix '\0' in output file
  • 57eb179 Make a fix_subtitles function
  • e3e810f Fix bug with asterisk

📊 Changes

15 files changed (+259 additions, -196 deletions)

View changed files

📝 src/lib_ccx/ccx_common_common.c (+7 -6)
📝 src/lib_ccx/ccx_common_common.h (+1 -1)
📝 src/lib_ccx/ccx_common_option.c (+4 -2)
📝 src/lib_ccx/ccx_common_option.h (+4 -2)
📝 src/lib_ccx/ccx_encoders_common.c (+5 -6)
📝 src/lib_ccx/ccx_encoders_common.h (+2 -1)
📝 src/lib_ccx/ccx_encoders_helpers.c (+121 -92)
📝 src/lib_ccx/ccx_encoders_helpers.h (+8 -6)
📝 src/lib_ccx/ccx_encoders_smptett.c (+1 -5)
📝 src/lib_ccx/ccx_encoders_srt.c (+3 -6)
📝 src/lib_ccx/ccx_encoders_ssa.c (+2 -5)
📝 src/lib_ccx/ccx_encoders_transcript.c (+3 -5)
📝 src/lib_ccx/configuration.c (+1 -0)
📝 src/lib_ccx/params.c (+85 -59)
📝 src/lib_ccx/params_dump.c (+12 -0)

📄 Description

Fix #1114

  • 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.
  • I have used CCExtractor just a couple of times.
  • I have mentioned this change in the changelog.

This PR reformats most of the capitalization parts as well to make a better interface.

This PR also removes one of the list that was used for capitalization before spell_lower as it was useless.

PS: also btw, for the list of words, I took them from Wikipedia, I don't have that good of a vocab 😂


🔄 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/1139 **Author:** [@NilsIrl](https://github.com/NilsIrl) **Created:** 12/8/2019 **Status:** ✅ Merged **Merged:** 1/5/2020 **Merged by:** [@canihavesomecoffee](https://github.com/canihavesomecoffee) **Base:** `master` ← **Head:** `filter_bad_words` --- ### 📝 Commits (10+) - [`5b29db3`](https://github.com/CCExtractor/ccextractor/commit/5b29db341f15dc2ca02e7c3daac2d6e9d34f624d) Remove space before ';' - [`99a12b8`](https://github.com/CCExtractor/ccextractor/commit/99a12b873717df0ec97a15d3f2c33acf7bef6c97) Add --kf option and parse files - [`7d8499a`](https://github.com/CCExtractor/ccextractor/commit/7d8499a7fbece2386a1784e2107d7462a6854eef) Rename profanity_file to filter_profanity_file. Dump params - [`a7d2264`](https://github.com/CCExtractor/ccextractor/commit/a7d2264cc1c4228863f305ff35c30c7e70153892) Use correct function - [`2739602`](https://github.com/CCExtractor/ccextractor/commit/27396025755bf96f9913002d5fe1bcd3e2d9d0ea) Add missing continue - [`8ef89f6`](https://github.com/CCExtractor/ccextractor/commit/8ef89f6bf18bf14f6c7a663e9e9e514a6242a156) Fix double free error - [`59a8c7a`](https://github.com/CCExtractor/ccextractor/commit/59a8c7a049185e3fe25c6efa08eb3901fb5fd47f) Censor word when in dictionary - [`37e4d41`](https://github.com/CCExtractor/ccextractor/commit/37e4d4163f64716a8d2c314a2310647a93547701) Fix '\0' in output file - [`57eb179`](https://github.com/CCExtractor/ccextractor/commit/57eb1795aa32cdbcaafaa03295aac426b88d1ab3) Make a fix_subtitles function - [`e3e810f`](https://github.com/CCExtractor/ccextractor/commit/e3e810f34eb6d3065c8deda1871448721db0ad5d) Fix bug with asterisk ### 📊 Changes **15 files changed** (+259 additions, -196 deletions) <details> <summary>View changed files</summary> 📝 `src/lib_ccx/ccx_common_common.c` (+7 -6) 📝 `src/lib_ccx/ccx_common_common.h` (+1 -1) 📝 `src/lib_ccx/ccx_common_option.c` (+4 -2) 📝 `src/lib_ccx/ccx_common_option.h` (+4 -2) 📝 `src/lib_ccx/ccx_encoders_common.c` (+5 -6) 📝 `src/lib_ccx/ccx_encoders_common.h` (+2 -1) 📝 `src/lib_ccx/ccx_encoders_helpers.c` (+121 -92) 📝 `src/lib_ccx/ccx_encoders_helpers.h` (+8 -6) 📝 `src/lib_ccx/ccx_encoders_smptett.c` (+1 -5) 📝 `src/lib_ccx/ccx_encoders_srt.c` (+3 -6) 📝 `src/lib_ccx/ccx_encoders_ssa.c` (+2 -5) 📝 `src/lib_ccx/ccx_encoders_transcript.c` (+3 -5) 📝 `src/lib_ccx/configuration.c` (+1 -0) 📝 `src/lib_ccx/params.c` (+85 -59) 📝 `src/lib_ccx/params_dump.c` (+12 -0) </details> ### 📄 Description Fix #1114 - [X] I have read and understood the [contributors guide](https://github.com/CCExtractor/ccextractor/blob/master/.github/CONTRIBUTING.md). - [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. - [X] I have used CCExtractor just a couple of times. - [ ] **I have mentioned this change in the [changelog](https://github.com/CCExtractor/ccextractor/blob/master/docs/CHANGES.TXT).** This PR reformats most of the capitalization parts as well to make a better interface. This PR also removes one of the list that was used for capitalization before `spell_lower` as it was useless. PS: also btw, for the list of words, I took them from Wikipedia, I don't have that good of a vocab 😂 --- <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:19:14 +00:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/ccextractor#1915