From 6fde557864505b470c438e4161ee494f29b90d63 Mon Sep 17 00:00:00 2001 From: Carl Hetherington Date: Mon, 15 Feb 2016 12:25:35 +0000 Subject: [PATCH] Use get_pixel_size rather than get_size which is slightly nicer as we then don't need PANGO_SCALE. --- src/lib/render_subtitles.cc | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/src/lib/render_subtitles.cc b/src/lib/render_subtitles.cc index eb67d68a5..dd8e8870b 100644 --- a/src/lib/render_subtitles.cc +++ b/src/lib/render_subtitles.cc @@ -240,7 +240,7 @@ render_line (list subtitles, list > fonts, int layout_width; int layout_height; - layout->get_size (layout_width, layout_height); + layout->get_pixel_size (layout_width, layout_height); int x = 0; switch (subtitles.front().h_align ()) { @@ -250,11 +250,11 @@ render_line (list subtitles, list > fonts, break; case dcp::HALIGN_CENTER: /* h_position is distance between centre of frame and centre of subtitle */ - x = (0.5 + subtitles.front().h_position()) * target.width - layout_width / (PANGO_SCALE * 2); + x = (0.5 + subtitles.front().h_position()) * target.width - layout_width / 2; break; case dcp::HALIGN_RIGHT: /* h_position is distance between right of frame and right of subtitle */ - x = (1.0 - subtitles.front().h_position()) * target.width - layout_width / PANGO_SCALE; + x = (1.0 - subtitles.front().h_position()) * target.width - layout_width; break; } @@ -267,15 +267,15 @@ render_line (list subtitles, list > fonts, 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; + y = subtitles.front().v_position() * target.height - layout_height; break; case dcp::VALIGN_CENTER: /* v_position is distance between centre of frame and centre of subtitle */ - y = (0.5 + subtitles.front().v_position()) * target.height - layout_height / (PANGO_SCALE * 2); + y = (0.5 + subtitles.front().v_position()) * target.height - layout_height / 2; break; case dcp::VALIGN_BOTTOM: /* v_position is distance between bottom of frame and bottom of subtitle */ - y = (1.0 - subtitles.front().v_position()) * target.height - layout_height / PANGO_SCALE; + y = (1.0 - subtitles.front().v_position()) * target.height - layout_height; break; } -- 2.30.2