From e9edb9ffd2deee65b2ab044a4a8bc149ed44a0b4 Mon Sep 17 00:00:00 2001 From: Carl Hetherington Date: Fri, 20 Nov 2015 00:03:14 +0000 Subject: Adjust subtitle position when referenced from top. SMPTE says "top" means that vposition is from the top of the screen to the top of the subtitle, but Dolby servers apparently make it top of screen to bottom of subtitle; adjust the preview to do the latter. --- src/lib/render_subtitles.cc | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) (limited to 'src') diff --git a/src/lib/render_subtitles.cc b/src/lib/render_subtitles.cc index 9b5f585e5..26f41c89e 100644 --- a/src/lib/render_subtitles.cc +++ b/src/lib/render_subtitles.cc @@ -261,8 +261,13 @@ render_line (list subtitles, list > fonts, int y = 0; switch (subtitles.front().v_align ()) { case dcp::VALIGN_TOP: - /* v_position is distance between top of frame and top of subtitle */ - y = subtitles.front().v_position() * target.height; + /* SMPTE says that v_position is the distance between top + of frame and top of subtitle, but this doesn't always seem to be + the case in practice; Gunnar Ásgeirsson's Dolby server appears + to put VALIGN_TOP subs with v_position as the distance between top + of frame and bottom of subtitle. + */ + y = subtitles.front().v_position() * target.height - layout_height / PANGO_SCALE; break; case dcp::VALIGN_CENTER: /* v_position is distance between centre of frame and centre of subtitle */ -- cgit v1.2.3