diff options
| author | John Stebbins <[email protected]> | 2017-03-10 12:56:55 -0700 |
|---|---|---|
| committer | GitHub <[email protected]> | 2017-03-10 12:56:55 -0700 |
| commit | bd22891d633e16d7aedddadd2e4dd716f615d847 (patch) | |
| tree | 4194d0fd483759f8e8f5226ac479e5ad193a9ef9 /libhb/work.c | |
| parent | 1f6c91c00be0f8f47de1d5c77b945ad6ecd50ec5 (diff) | |
subtitles: simplify and shorten subtitle descriptions (#591)
* subtitles: simplify and shorten subtitle descriptions
Generally, it eliminates parens to make things more readable.
I.e. it turns this:
English (Closed Caption)(Wide Screen)(Bitmap)(VOBSUB)
Into this:
English, Closed Caption [Wide Screen, VOBSUB]
* Revise punctuation per BradleyS request
* fix subtitle description formatting
* incorporate suggestions from PR
Diffstat (limited to 'libhb/work.c')
| -rw-r--r-- | libhb/work.c | 34 |
1 files changed, 19 insertions, 15 deletions
diff --git a/libhb/work.c b/libhb/work.c index 5af53cb5d..1358bc12e 100644 --- a/libhb/work.c +++ b/libhb/work.c @@ -540,29 +540,33 @@ void hb_display_job_info(hb_job_t *job) { if( job->indepth_scan ) { - hb_log( " + subtitle, %s (track %d, id 0x%x) %s [%s]", + hb_log( " + subtitle, %s (track %d, id 0x%x, %s)", subtitle->lang, subtitle->track, subtitle->id, - subtitle->format == PICTURESUB ? "Picture" : "Text", - hb_subsource_name( subtitle->source ) ); + subtitle->format == PICTURESUB ? "Picture" : "Text"); } else if( subtitle->source == SRTSUB ) { /* For SRT, print offset and charset too */ - hb_log( " * subtitle track %d, %s (track %d, id 0x%x) Text [SRT] -> %s%s, offset: %"PRId64", charset: %s", - subtitle->out_track, subtitle->lang, subtitle->track, subtitle->id, - subtitle->config.dest == RENDERSUB ? "Render/Burn-in" : "Passthrough", - subtitle->config.default_track ? ", Default" : "", - subtitle->config.offset, subtitle->config.src_codeset ); + hb_log(" * subtitle track %d, %s (track %d, id 0x%x, Text) -> " + "%s%s, offset: %"PRId64", charset: %s", + subtitle->out_track, subtitle->lang, subtitle->track, + subtitle->id, + subtitle->config.dest == RENDERSUB ? "Render/Burn-in" + : "Passthrough", + subtitle->config.default_track ? ", Default" : "", + subtitle->config.offset, subtitle->config.src_codeset); } else { - hb_log( " * subtitle track %d, %s (track %d, id 0x%x) %s [%s] -> %s%s%s", - subtitle->out_track, subtitle->lang, subtitle->track, subtitle->id, - subtitle->format == PICTURESUB ? "Picture" : "Text", - hb_subsource_name( subtitle->source ), - subtitle->config.dest == RENDERSUB ? "Render/Burn-in" : "Passthrough", - subtitle->config.force ? ", Forced Only" : "", - subtitle->config.default_track ? ", Default" : "" ); + hb_log(" * subtitle track %d, %s (track %d, id 0x%x, %s) -> " + "%s%s%s", + subtitle->out_track, subtitle->lang, subtitle->track, + subtitle->id, + subtitle->format == PICTURESUB ? "Picture" : "Text", + subtitle->config.dest == RENDERSUB ? "Render/Burn-in" + : "Passthrough", + subtitle->config.force ? ", Forced Only" : "", + subtitle->config.default_track ? ", Default" : "" ); } } } |