summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorwm4 <wm4@nowhere>2016-09-07 13:42:55 +0200
committerwm4 <wm4@nowhere>2016-09-07 13:42:55 +0200
commit2f1eb49e9f130c728c09a3458ff8449bc571c0c0 (patch)
tree17b26ffb08ed745c666b67561f54baeb34766d53
parent355361e1e36e995bb2650a65f7ccf6ea1a30daf1 (diff)
downloadmpv-2f1eb49e9f130c728c09a3458ff8449bc571c0c0.tar.bz2
mpv-2f1eb49e9f130c728c09a3458ff8449bc571c0c0.tar.xz
sub: actually apply text alignment options to non-ASS subtitles
This is a bug fix, and the text alignment functionality probably got lost sometime along the way. For ASS subtitles, this could have unintended consequences, so it's hard to get right - thus it's not applied to ASS subtitles. For other text subtitles, this should be fine, though. It still works on ASS subtitles as promised by the manpage if --no-sub-ass is used.
-rw-r--r--sub/sd_ass.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/sub/sd_ass.c b/sub/sd_ass.c
index 34a49c1501..6bbd701998 100644
--- a/sub/sd_ass.c
+++ b/sub/sd_ass.c
@@ -329,6 +329,8 @@ static void configure_ass(struct sd *sd, struct mp_osd_res *dim,
set_force_flags |= ASS_OVERRIDE_BIT_STYLE | ASS_OVERRIDE_BIT_FONT_SIZE;
if (opts->ass_style_override == 4)
set_force_flags |= ASS_OVERRIDE_BIT_FONT_SIZE;
+ if (converted)
+ set_force_flags |= ASS_OVERRIDE_BIT_ALIGNMENT;
ass_set_selective_style_override_enabled(priv, set_force_flags);
ASS_Style style = {0};
mp_ass_set_style(&style, 288, opts->sub_text_style);