Check for playlist directory on new playlist, rather than save.

Add bind().

Set Delete button sensitivity correctly.

Extract selected().

Setup editor sensitivity on startup.

Don't display first playlist on startup.

Save playlist when content is added, moved or removed.

Remove unnecessary non-const operator[].

Add and use SPL::swap().

Cleanup: white space.

Generalise the Changed message from SPL.

Escape entities just before Pango rendering (#2382).

Previously, text coming out of the player would have things like
& escaped to &.  This escaping is also done by libxml++ when
writing XML, so doing it in the player would mean it was done
twice.

We do, however, need to escape things before passing them to Pango
as otherwise it gives errors and renders nothing for the line.

Here we move the escaping to just before the rendering, meaning
that in the reset of DoM we should pass unescaped strings around.

Add some failing tests.

Supporters update.

Add default release territory configuration (#2375).

Extract RegionSubtagWidget.

Add some comments.

Add default audio language configuration (#2375).

Remove default container configuration option.

I think this is basically useless now that DoM guesses a good container
when the first video is added.  It made more sense before we did that.

Supporters update.

Return AVERROR_EOF from the avio_read method when appropriate.

Fix audio glitches when merging resampled audio (#2373).

Noisy change to get film into AudioContent::modify_trim_start().

Remove some backwards compatibility that will cause problems in an upcoming commit.

Add another test, including a frame rate change.

Give a slightly better error message in a failing test.

Add failing test for #2373.

_stream_states must be reset on seek otherwise #2368.

Fix sound corruption with multiple sound files (#2368).

last_push_end would be initialised to the position of the content,
meaning that content at (for example) 60s into the project would
be treated as the leader, and content much earlier would then be
considered finished by the code which checks ignore_streams_behind
(as its last push was more then 5s before the leader).

This caused audio corruption, shown up by the test here.

Cleanup: use dcp_file().

More warning fixes.

More warning fixes.

Fix more warnings.

Fix warnings.

Allow a slightly worrying amount of error so that the Windows tests
that rely on font metrics still pass.

Bump libdcp for new equality option for subtitles.

Extract RegionSubtagDialog to its own files.

Extract LanguageSubtagPanel to its own files.

Extract SubtagListCtrl to its own files.

Fix player config window on macOS (#2376).

Fix crash when adding cinemas while a search is in force (#2378).

Cleanup: extract sorted_cinemas().

Cleanup: const correctness.

Cleanup: add convert_to_lower() and matches_search().

Be a bit more defensive in a test that is failing on Windows.

Bump private git so that we use Liberation Sans rather than Arial for a subtitle test.

Only check test private data if --check is passed.

Check that the private test data repo is as we expect before running tests on Linux/macOS.

Fix some missing test reference updates for vpos changes.

Fix test/data detachment SNAFU.

Fix subtitle vertical position (#2367).

Previously we would not account for the differences in what vertical
position means between Interop and SMPTE.  For interop, vertical
position is the distance from the reference point to the text
baseline, whereas for SMPTE it is the distance from the reference
point to the top/middle/bottom of the subtitle (depending on the
reference).

This caused differences between the preview and the DCP for some
cases (notably, using SRT/SSA and making Interop DCPs, or converting
Interop DCP subs to SMPTE, or vice versa).

Click on the timeline time axis view moves the playhead (#2369).

Fix TimelineTimeAxisView::bbox() to be correct.

Move y_pos to TimelineView.

Remove unused line.

Bundle dcpkdm from libdcp as dcpomatic2_kdm_inspect (#2366).

Supporters update.

Add comment to try to clarify _dcp_content_types a little for translators.

Add David Perrenoud to translators list.

Updated fr_FR translation from David Perrenoud.

Improve flushing behaviour when there is a lot of space to fill (#2364).

Previously a call to flush() could result in a lot of audio being
emitted from the decoder (if there is a big gap between the end
of the audio and the video).  This would end up being emitted in
one chunk from the player, crashing the audio analyser with an OOM
in some cases.

Clarify one log message and add another.

Add failing test for #2364.

Basic WebVTT subtitle support (#1361).

Cleanup: whitespace.

Fix macOS build.

Bump libsub for fix unicode LINE SEPARATOR in subtitles.

Fix uninitialised variable.

Fix error message.

Cleanup: J2KEncoder does not need enable_shared_from_this.

Cleanup: Player does not need enable_shared_from_this.

Cleanup: remove unnecessary explicit.

Cleanup: use default value in header.

Cleanup: better variable name.

Hide the endless stream of warnings from wx/GTK.

Extract ImageStore.

Strictly I think we should be putting each component of an image into _pending_images.
We probably get away with only keeping component 0 but I think that could perhaps lead
to use-after-free as the Image for components 1 and 2 could go away a bit before
it should.

Fix build on macOS and Windows.

Bump libdcp for subtitle Z-position fixes (#2356).

Fix test references for passive option change.

Cleanup: use new CheckBox::bind().

Enable some debug logging for TMS upload.

Add passive mode option to TMS upload.

Disabling this fixes TMS upload with some FTP servers
(reported on a Synology NAS).

Add get() and bind() to CheckBox.

Throw exception if av_frame_alloc() fails.

Add tooltip to player DCP name with path details.

Fix player with high-frame-rate DCPs.

Supporters update.

Fix player being destroyed while the butler is still using it.

Remove unnecessary copy constructors (default ones are OK).

Allow move construction of Player.

Allow move construction of AudioMerger.

Allow move construction of ActiveText.

Further test/data SNAFU.

Fix mangling of the v2.16.x test/data commit somewhere along the line.

Hopefully fix crash when the same frame is encoded twice.

Previously if some frame X was passed to the encoder twice
I think this would happen:

X1 written; X stored to _pending_images
X2 written; _pending_images remains the same
X1 encode finishes; X is discarded from _pending_images
Data for X2 is read by the encoder but was just freed

I think this might have resulted in intermittent crashes, which were
fairly common in the 2.17.x branch with the test
ffmpeg_encoder_prores_regression_1

But I didn't conclusively prove it.

Fix missing switch of Append -> add (#2358).

Add Daniel Courville to testers.

Fix colour range property for subsampled sources (#2357).

Fix macOS 10.8 build.