From: Carl Hetherington <cth@carlh.net>
Date: Fri, 20 Jan 2023 22:53:54 +0000 (+0100)
Subject: Use bounding_box() instead of render_text() for speed.
X-Git-Tag: v2.16.45~18^2~11
X-Git-Url: https://git.carlh.net/gitweb/?p=dcpomatic.git;a=commitdiff_plain;h=f40fa28e3ee886c160d4ced26e4e9322d7ece5f0;ds=sidebyside

Use bounding_box() instead of render_text() for speed.
---

diff --git a/src/lib/analyse_subtitles_job.cc b/src/lib/analyse_subtitles_job.cc
index d222ad3b8..f140f13d1 100644
--- a/src/lib/analyse_subtitles_job.cc
+++ b/src/lib/analyse_subtitles_job.cc
@@ -112,10 +112,10 @@ AnalyseSubtitlesJob::analyse(PlayerText const& text, TextType type)
 
 	/* We can provide dummy values for time and frame rate here as they are only used to calculate fades */
 	dcp::Size const frame = _film->frame_size();
-	for (auto i: render_text(text.string, frame, dcpomatic::DCPTime(), 24)) {
+	for (auto i: bounding_box(text.string, frame)) {
 		dcpomatic::Rect<double> rect (
-			double(i.position.x) / frame.width, double(i.position.y) / frame.height,
-			double(i.image->size().width) / frame.width, double(i.image->size().height) / frame.height
+			double(i.x) / frame.width, double(i.y) / frame.height,
+			double(i.width) / frame.width, double(i.height) / frame.height
 			);
 		if (!_bounding_box) {
 			_bounding_box = rect;