Skip to content

Commit

Permalink
[IMPROVEMENT] Clarify CEA-608/708 Subtitle Extraction Behavior #1448 (#…
Browse files Browse the repository at this point in the history
…1663)

* made changes to the file src/lib_ccx/params.c

* made changes to the help message.
  • Loading branch information
tank0nf authored Feb 14, 2025
1 parent b08c5fa commit ffe075b
Show file tree
Hide file tree
Showing 2 changed files with 19 additions and 7 deletions.
20 changes: 13 additions & 7 deletions src/lib_ccx/params.c
Original file line number Diff line number Diff line change
Expand Up @@ -969,17 +969,17 @@ void print_usage(void)
mprint(" You can adjust, or disable, the algorithm settings with the following\n");
mprint(" parameters.\n");
mprint("\n");
mprint("Notes on times:\n"
" --startat and --endat times are used first, then -delay.\n");
mprint("Notes on times:\n");
mprint(" --startat and --endat times are used first, then -delay.\n");
mprint(" So if you use --srt -startat 3:00 --endat 5:00 --delay 120000, ccextractor will\n");
mprint(" generate a .srt file, with only data from 3:00 to 5:00 in the input file(s)\n");
mprint(" and then add that (huge) delay, which would make the final file start at\n");
mprint(" 5:00 and end at 7:00.\n");
mprint("\n");
mprint("Notes on codec options:\n");
mprint(" If codec type is not selected then first elementary stream suitable for \n"
" subtitle is selected, please consider --teletext -noteletext override this\n"
" option.\n");
mprint(" If codec type is not selected then first elementary stream suitable for\n");
mprint(" subtitle is selected, please consider --teletext --noteletext override this\n");
mprint(" option.\n");
mprint(" no-codec and codec parameter must not be same if found to be same \n"
" then parameter of no-codec is ignored, this flag should be passed \n"
" once, more then one are not supported yet and last parameter would \n"
Expand All @@ -992,8 +992,14 @@ void print_usage(void)
mprint(" The start window must be between the times given and must have enough time\n");
mprint(" to display the message for at least the specified time.\n");
mprint("\n");
mprint("Notes on the CEA-708 decoder:\n"
" While it is starting to be useful, it's\n");
mprint("Notes on the CEA-708 decoder:\n");
mprint(" By default, ccextractor now extracts both CEA-608 and CEA-708 subtitles\n");
mprint(" if they are present in the input. This results in two output files: one\n");
mprint(" for CEA-608 and one for CEA-708.\n");
mprint(" To extract only CEA-608 subtitles, use -1, -2, or -12.\n");
mprint(" To extract only CEA-708 subtitles, use -svc.\n");
mprint(" To extract both CEA-608 and CEA-708 subtitles, use both -1/-2/-12 and -svc.\n\n");
mprint(" While it is starting to be useful, it's\n");
mprint(" a work in progress. A number of things don't work yet in the decoder\n");
mprint(" itself, and many of the auxiliary tools (case conversion to name one)\n");
mprint(" won't do anything yet. Feel free to submit samples that cause problems\n");
Expand Down
6 changes: 6 additions & 0 deletions src/rust/src/args.rs
Original file line number Diff line number Diff line change
Expand Up @@ -104,6 +104,12 @@ Notes on adding credits:
to display the message for at least the specified time.
Notes on the CEA-708 decoder:
By default, ccextractor now extracts both CEA-608 and CEA-708 subtitles
if they are present in the input. This results in two output files: one
for CEA-608 and one for CEA-708.
To extract only CEA-608 subtitles, use -1, -2, or -12.
To extract only CEA-708 subtitles, use -svc.
To extract both CEA-608 and CEA-708 subtitles, use both -1/-2/-12 and -svc.
While it is starting to be useful, it's
a work in progress. A number of things don't work yet in the decoder
itself, and many of the auxiliary tools (case conversion to name one)
Expand Down

0 comments on commit ffe075b

Please sign in to comment.