summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorCarl Hetherington <cth@carlh.net>2023-11-10 23:51:46 +0100
committerCarl Hetherington <cth@carlh.net>2023-11-17 00:24:35 +0100
commit2603a529a05905f133bd52271fb1298e9a4a4aa5 (patch)
treea5228a95d93ca9d077e37c41a88c919086bf263a /src
parent7a877ff976b119a39797c1f6e8477b92dfbb30ca (diff)
Always add a font with an empty ID (#2649).
It's not just subtitle files with no <LoadFont> that can have subtitles without a specified font. In particular, DoM makes a single space subtitle with no font spec when it creates filler subtitles in multi-reel DCPs.
Diffstat (limited to 'src')
-rw-r--r--src/lib/dcp_examiner.cc4
-rw-r--r--src/lib/dcp_subtitle_content.cc4
-rw-r--r--src/lib/font_id_allocator.cc4
3 files changed, 3 insertions, 9 deletions
diff --git a/src/lib/dcp_examiner.cc b/src/lib/dcp_examiner.cc
index 0f9ae9544..ca77f2db3 100644
--- a/src/lib/dcp_examiner.cc
+++ b/src/lib/dcp_examiner.cc
@@ -211,9 +211,7 @@ DCPExaminer::DCPExaminer (shared_ptr<const DCPContent> content, bool tolerant)
for (auto const& font: asset->font_data()) {
_fonts.push_back({reel_index, asset->id(), make_shared<dcpomatic::Font>(font.first, font.second)});
}
- if (asset->font_data().empty()) {
- _fonts.push_back({reel_index, asset->id(), make_shared<dcpomatic::Font>("")});
- }
+ _fonts.push_back({reel_index, asset->id(), make_shared<dcpomatic::Font>("")});
}
_text_count[TextType::CLOSED_CAPTION] = std::max(_text_count[TextType::CLOSED_CAPTION], static_cast<int>(reel->closed_captions().size()));
diff --git a/src/lib/dcp_subtitle_content.cc b/src/lib/dcp_subtitle_content.cc
index e9db5546c..b3e24d5e2 100644
--- a/src/lib/dcp_subtitle_content.cc
+++ b/src/lib/dcp_subtitle_content.cc
@@ -84,9 +84,7 @@ DCPSubtitleContent::examine (shared_ptr<const Film> film, shared_ptr<Job> job)
}
}
- if (only_text()->fonts().empty()) {
- only_text()->add_font(make_shared<Font>(""));
- }
+ only_text()->add_font(make_shared<Font>(""));
}
DCPTime
diff --git a/src/lib/font_id_allocator.cc b/src/lib/font_id_allocator.cc
index ef25dc642..70eda2b06 100644
--- a/src/lib/font_id_allocator.cc
+++ b/src/lib/font_id_allocator.cc
@@ -63,9 +63,7 @@ FontIDAllocator::add_fonts_from_asset(int reel_index, shared_ptr<const dcp::Subt
_map[Font(reel_index, asset->id(), font.first)] = 0;
}
- if (asset->font_data().empty()) {
- _map[Font(reel_index, asset->id(), "")] = 0;
- }
+ _map[Font(reel_index, asset->id(), "")] = 0;
}