Test updates now that there can't be <1s long DCPs any more.

Coalesce short reels.

Improve and fix some tests.

Enforce a minimum DCP length of 1 second.

Get some better errors when verification-based tests fail.

Enforce minimum reel length when using REELTYPE_BY_LENGTH.

Add a test for short pieces of content amongst longer.

Failing test for reels with a short length (in bytes) specified.

Add a (failing) test for reels created between two other
pieces of content.

Move private_data into TestPaths

Add failing test for one short-reel possibility.

Give a better error when opening a DCP with File -> Open by mistake (#1723).

Merge branch 'v2.15.x' of ssh://git.carlh.net/home/carl/git/dcpomatic into v2.15.x

Make DKDMs with validity 2 minutes shorter than the signer certificate
validity (#1728).

Add missing icons for _playlist to the macOS build.

Fix test failure with changes to thread management.

Stop threads at the start of their object's destruction in all Job cases.

Fix crash in verification due to mismatched wxString::Format placeholder.

Fix weirdly-allowed misplaced bracket.

Remove unused LOG_DEBUG_DECODE and re-use it to add some 3D shuffler logging.

Missing patron.

Call wxInitAllImageHandlers and fix Playlist Editor AppName.

Fix uninitialised variable.

Fix loading of tick/no_tick graphics for playlist editor on Windows.

Fix broken test.

Support playlists in the player in dual-screen mode.

Separate SPL from a version which signals when things change.

Add some comments.

Make DMG for playlist editor on macOS.

Add 'proper' config dialog for the playlist editor.

Fix title of player configuration dialogue.

Various playlist editor developments and fixes.

Copy swaroop_ playlist editor stuff back to main DoM.

Move swaroop playlist editor stuff into its own swaroop_
files.

Allow changing colour conversion settings for multiple pieces of content at the same time (github #7).

Fix exceptions on some metadata.xml.

Fix incorrect results when applying some timing operations to
more than one piece of content at once (e.g. trim start).

Allow > 0dB gain per channel in the audio matrix (#1720).

Use bound arguments to lose some methods.

Add and use dB/linear conversion functions.

Missing include.

Fix incorrect tab-to-space.

Tweak DMG creation for Apple changes.

Apply FFmpeg palette fix to examination too; fix build with old FFmpeg.

Another macOS std::list boost::thread SNAFU.

Can't put boost::thread in a std::list with our macOS compiler.

Merge branch 'v2.15.x' of ssh://git.carlh.net/home/carl/git/dcpomatic into v2.15.x

It may not be allowed to request a zero timeout on macOS.

Nicer fix for 2D-labelled-3D checking from master.

Remove incorrect comment.

Write logs during tests to a file.

Remove unused variable.

Make a new AudioBuffers constructor and use it to avoid some use of the more complicated copy_from.

Better errors if butler doesn't give video during export.

New test bits.

Just having no specified lines value does not mean we definitely need place the subtitle ourselves.

Nudge L/R-aligned subs in from the edge of the screen; do our own placement of subs with no screen height in lines specified.

Rename and extend check_packets.py

Fix crash.

Fix crash with bitmapped subs that have zero width or height.

Add new script.

Various improvements to make_dummy_files

Various thread cleanups.

Use a non-pointer boost::thread and a std::atomic for the stop flag.

Missing #include for previous.

Fix deadlock during content examination.

Before this fix, the following situation could happen in threads
A and B:

A: Some DONE signal happens; this triggers setup_pieces which
   takes a lock on the player mutex.

B: FFmpegContent::examine takes a lock on the content mutex.
B: FFmpegContent::examine adds a stream
B: That causes STREAMS PENDING to be emitted.
B: This tries to take a lock on the player mutex so it can update _suspended

A: setup_pieces tries to access some content information, hence
   tries to take a lock on the content mutex.

Now B is holding the CL and awaiting the PL and A is holding
the PL and awaiting the CL.

It feels like the root cause of this is that while setup_pieces
is happening another change (which would itself cause setup_pieces)
is announced, and this isn't dealt with properly.

There are two steps here; _suspended is protected with an atomic
rather than using _mutex, and also it can cope with being updated
recursively.

Fix crash with sub-sample push parts in AudioMerger.

Forward-ported from b86b15391074a68149f8c4a51958c7873d74def0 in master.

Fix missing preferences icon on macOS.

Warn if doing a gain calculation might cause clipping.

Apply fader-based gain calculations to the current gain, not 0dB.

Another thinko fix.

Fix thinko two commits ago.

Cache film length for Player::pass.

Use memset for zeroing float buffers; it's faster, and I think
we get away with it on all the platforms we care about.

Support binary STL subtitle files.

Add patron.

Make FileError say what path the problem was with.

Add some PT-style names to the guesswork on which channel a particular sound file belongs to.

Stop invalid dates causing boost::posix_time to raise exceptions.

swaroop variant build fix.

Don't truncate active_video_frame_rate to an int otherwise setting fade on non-integer-rate sources behaves oddly.

Fix deadlock.

Give DCPDecoder its own ::position which just returns its internal
_next time.

This is important because Decoder::position does the wrong thing
with DCPs in the following case.

1.  DCPDecoder emits a subtitle event (start/stop) at time t.
2.  There follows a long time T with no subtitle events.  During
    this time the DCPDecoder's position is reported as t (since
    TextDecoder notes its position as the time of the last thing
    it emitted --- which is all it reasonably can do, I think).
3.  During this T the DCPDecoder may be incorrectly pass()ed because
    its position is reported as earlier than it really is; this results
    in video/audio being emitted by the DCPDecoder but other contemporary
    sources may not be pass()ed.

The upshot of this can be that no audio is emitted, as a contemporary audio
source is not pass()ed and hence the merger is waiting for audio that will
take a long time to come.  When the butler is running this can result in
audio underruns as the video buffers overflow with no sign of any audio.

It is also simpler this way; DCPDecoder was already maintaining the required
information.

Update text panel 'why not' note when film's standard is changed.

Forward-ported from e8582393bd1367fff36bae8e053d799073d8b2ed in master.

Make the 'why not' messages for VF a bit clearer.

Before it sounded like just because we couldn't refer to (e.g.) the audio
we couldn't refer to any part of the DCP.

Merge a set of changes which run the OpenGL video updates in a separate
thread, hopefully making things more elegant and robust.

First version of player stress-test management script.

Some comments.

Some simple-video-view tidy-ups.

Fix problems when starting to play after a seek, if the start
of playback happens while we are awaiting an idle handler to
go through to update the view after the seek.

If this idle handler has not gone through the viewer will report
an incorrect position, which is used by the FilmViewer to start
the audio stream at the wrong time.

Use glTexSubImage2D when possible, as suggested by https://khronos.org/opengl/wiki/Common_Mistakes

Basic stress-test function for player.

Don't busy-wait when there's nothing to play.

Set up things correctly on initialisation.

Fix event handling (hopefully).

Add assert.

Check that get_next_frame succeeds.

Fix some crashes.

More tidying up.

Various cleanups and thread-safety.