From f0c3c3830257ee7281ea9c5e5d49feb228a2fa20 Mon Sep 17 00:00:00 2001 From: Carl Hetherington Date: Sun, 28 May 2023 02:12:39 +0200 Subject: Don't unconditionally clear _fonts when inspecting assets. --- src/interop_subtitle_asset.cc | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) (limited to 'src') diff --git a/src/interop_subtitle_asset.cc b/src/interop_subtitle_asset.cc index 0116c11c..d0c80f5e 100644 --- a/src/interop_subtitle_asset.cc +++ b/src/interop_subtitle_asset.cc @@ -230,8 +230,6 @@ InteropSubtitleAsset::write (boost::filesystem::path p) const void InteropSubtitleAsset::resolve_fonts (vector> assets) { - _fonts.clear(); - for (auto asset: assets) { auto font = dynamic_pointer_cast(asset); if (!font) { @@ -243,7 +241,12 @@ InteropSubtitleAsset::resolve_fonts (vector> assets) for (auto load_font_node: _load_font_nodes) { auto const path_in_load_font_node = _file->parent_path() / load_font_node->uri; if (font->file() && path_in_load_font_node == *font->file()) { - _fonts.push_back(Font(load_font_node->id, asset->id(), font->file().get())); + auto existing = std::find_if(_fonts.begin(), _fonts.end(), [load_font_node](Font const& font) { return font.load_id == load_font_node->id; }); + if (existing != _fonts.end()) { + *existing = Font(load_font_node->id, asset->id(), font->file().get()); + } else { + _fonts.push_back(Font(load_font_node->id, asset->id(), font->file().get())); + } } } } -- cgit v1.2.3