<feed xmlns='http://www.w3.org/2005/Atom'>
<title>dcpomatic/test/subtitle_font_id_test.cc, branch writev-debug</title>
<subtitle>DCP-o-matic DCP tools</subtitle>
<id>https://git.carlh.net/cgit/dcpomatic/atom?h=writev-debug</id>
<link rel='self' href='https://git.carlh.net/cgit/dcpomatic/atom?h=writev-debug'/>
<link rel='alternate' type='text/html' href='https://git.carlh.net/cgit/dcpomatic/'/>
<updated>2024-09-03T15:02:24Z</updated>
<entry>
<title>Adapt to some libdcp class renaming to allow open captions/closed subtitles.</title>
<updated>2024-09-03T15:02:24Z</updated>
<author>
<name>Carl Hetherington</name>
<email>cth@carlh.net</email>
</author>
<published>2024-08-06T21:57:22Z</published>
<link rel='alternate' type='text/html' href='https://git.carlh.net/cgit/dcpomatic/commit/?id=f3ab77e34a41ad4d8f5156e9d6598314975b9565'/>
<id>urn:sha1:f3ab77e34a41ad4d8f5156e9d6598314975b9565</id>
<content type='text'>
</content>
</entry>
<entry>
<title>Fix test build.</title>
<updated>2024-06-28T14:53:00Z</updated>
<author>
<name>Carl Hetherington</name>
<email>cth@carlh.net</email>
</author>
<published>2024-06-28T14:53:00Z</published>
<link rel='alternate' type='text/html' href='https://git.carlh.net/cgit/dcpomatic/commit/?id=6467d7542585b0d61f30c4a87ec93db1d52cc2f5'/>
<id>urn:sha1:6467d7542585b0d61f30c4a87ec93db1d52cc2f5</id>
<content type='text'>
</content>
</entry>
<entry>
<title>Merge remote-tracking branch 'origin/main' into v2.17.x</title>
<updated>2024-06-28T11:04:30Z</updated>
<author>
<name>Carl Hetherington</name>
<email>cth@carlh.net</email>
</author>
<published>2024-06-28T11:04:30Z</published>
<link rel='alternate' type='text/html' href='https://git.carlh.net/cgit/dcpomatic/commit/?id=d70f72ad7b25f5b71e8c27381e8bfdce6c0fa5b4'/>
<id>urn:sha1:d70f72ad7b25f5b71e8c27381e8bfdce6c0fa5b4</id>
<content type='text'>
</content>
</entry>
<entry>
<title>Fix error when importing bad subtitle file (#2838).</title>
<updated>2024-06-27T18:14:31Z</updated>
<author>
<name>Carl Hetherington</name>
<email>cth@carlh.net</email>
</author>
<published>2024-06-27T18:14:31Z</published>
<link rel='alternate' type='text/html' href='https://git.carlh.net/cgit/dcpomatic/commit/?id=d6e8b54b2b60f70f1a8837183bc184809977be67'/>
<id>urn:sha1:d6e8b54b2b60f70f1a8837183bc184809977be67</id>
<content type='text'>
The subtitle XML refers to a font with no corresponding &lt;LoadFont&gt;.
</content>
</entry>
<entry>
<title>Simplify the FontIDAllocator a lot (#2827).</title>
<updated>2024-06-11T07:52:28Z</updated>
<author>
<name>Carl Hetherington</name>
<email>cth@carlh.net</email>
</author>
<published>2024-06-09T23:05:43Z</published>
<link rel='alternate' type='text/html' href='https://git.carlh.net/cgit/dcpomatic/commit/?id=a224ef26f48e1a9a9189c2b935abb06147b342bc'/>
<id>urn:sha1:a224ef26f48e1a9a9189c2b935abb06147b342bc</id>
<content type='text'>
This is at the expense of forward compatibility, and the need to
re-examine subtitle content (losing custom fonts as we do so).

But it does mean that the code is simpler, and there's not this weird
growth of IDs where a DCP gets imported with some font, and then the ID
becomes 0_font, and if you do it again it's 0_0_font, and so on.
</content>
</entry>
<entry>
<title>Fix an erroneous font ID error that can happen when loading projects from builds without release version tags.</title>
<updated>2024-06-09T21:38:24Z</updated>
<author>
<name>Carl Hetherington</name>
<email>cth@carlh.net</email>
</author>
<published>2024-06-09T21:38:24Z</published>
<link rel='alternate' type='text/html' href='https://git.carlh.net/cgit/dcpomatic/commit/?id=f8a6ce7ef5cc7e52aa10a6a133b4e628f3d2a50d'/>
<id>urn:sha1:f8a6ce7ef5cc7e52aa10a6a133b4e628f3d2a50d</id>
<content type='text'>
So that last_written_by_earlier_than() always returns true,
check_font_ids() happens when it doesn't really need to.
</content>
</entry>
<entry>
<title>Rename new_test_film2 -&gt; new_test_film.</title>
<updated>2024-05-22T08:33:45Z</updated>
<author>
<name>Carl Hetherington</name>
<email>cth@carlh.net</email>
</author>
<published>2024-05-20T14:55:57Z</published>
<link rel='alternate' type='text/html' href='https://git.carlh.net/cgit/dcpomatic/commit/?id=c95ba3eb99c5e4d6dca90cee7e5bb9077b6ed02c'/>
<id>urn:sha1:c95ba3eb99c5e4d6dca90cee7e5bb9077b6ed02c</id>
<content type='text'>
</content>
</entry>
<entry>
<title>Don't add a font with an empty ID for ccaps either (#2762).</title>
<updated>2024-02-11T22:31:39Z</updated>
<author>
<name>Carl Hetherington</name>
<email>cth@carlh.net</email>
</author>
<published>2024-02-11T19:13:04Z</published>
<link rel='alternate' type='text/html' href='https://git.carlh.net/cgit/dcpomatic/commit/?id=eb04ac87ccfa046dd342ca7b9e6478c3bdcabbba'/>
<id>urn:sha1:eb04ac87ccfa046dd342ca7b9e6478c3bdcabbba</id>
<content type='text'>
Previously this was all changed so that for subs we add the default
font in a cleaner way (in db22f81ccce9e1a5f205e6d8b3c0631fc039a173).

I can't see why we shouldn't treat ccaps the same way, and indeed
not doing it causes an error as shown by the included test.
</content>
</entry>
<entry>
<title>Inspect most DCPs made during tests with ClairMeta (#76).</title>
<updated>2024-01-15T11:44:53Z</updated>
<author>
<name>Carl Hetherington</name>
<email>cth@carlh.net</email>
</author>
<published>2024-01-12T14:29:12Z</published>
<link rel='alternate' type='text/html' href='https://git.carlh.net/cgit/dcpomatic/commit/?id=de1b4a967e6854782e88e83d48af74289977c8c3'/>
<id>urn:sha1:de1b4a967e6854782e88e83d48af74289977c8c3</id>
<content type='text'>
</content>
</entry>
<entry>
<title>Fix handling of empty font IDs and default DCP fonts (#2721) (part of #2722).</title>
<updated>2024-01-15T09:21:59Z</updated>
<author>
<name>Carl Hetherington</name>
<email>cth@carlh.net</email>
</author>
<published>2024-01-13T22:34:35Z</published>
<link rel='alternate' type='text/html' href='https://git.carlh.net/cgit/dcpomatic/commit/?id=db22f81ccce9e1a5f205e6d8b3c0631fc039a173'/>
<id>urn:sha1:db22f81ccce9e1a5f205e6d8b3c0631fc039a173</id>
<content type='text'>
Previously we used an empty font ID as the default for when a subtitle
has no Font, but in #2721 we saw a DCP with an empty font ID which
raised an assertion (because we'd already added our default font with
the empty ID).

Here we try to fix this (and also make the default font correctly be
that from the first &lt;LoadFont&gt;).
</content>
</entry>
</feed>
