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.

Remove some friends from FilmViewer.

Remove FilmViewer::refresh_view.

Move _state_timer into VideoView.

Accessor for ClosedCaptionsDialog.

Same thing with inter_size.

Move _inter_position out of FilmViewer.

Move _dropped into VideoView.

Small re-use cleanup.

Implement GLView::update.

Don't try opening error dialogues from the GL thread.

Fix display when there is no film.

Add comment.

Start thread when the window will have been created.  Fix logic error.

Use VideoView::one_video_frame.

Nicer protection of _player_video.  Always run GL thread rather than starting/stopping it.

Remove GL context accessor and hence lock.

Naming tweak.

Clean up access to stuff from Film.

A little thread safety.

Various timing hacks and development.

Remove PositionChanged in favour of consumers having their own GUI-thread timers.

Hack around changes in main branch with respect to suspend/resume.

Barely-functioning GL playback with new arrangement.

Add FIXME.

Add FilmViewer::time_until_next_frame.

Add accessor for butler.

Move _player_video into VideoView.

Move display_player_video into SimpleVideoView.

Move FilmViewer::get() into SimpleVideoView.

Remove temporary access to timer() method.

Remove redundant FilmViewer::video_position() and use FilmViewer::position() instead.

Use FilmViewer::{film,playing,position}()

Move ::timer into SimpleVideoView.

Fix typo causing failures to update the preview after changing
end trim (#1709 and probably #1434).

Extend the amount of allowed disagreement between the audio
coming from the content and the calculated periods of silence.

It sees that there is some scope for disagreements here when
trim start/end times are rounded to frame boundaries.

Fixes #1708.

Move credentials settings into certificate download dialogue (#1480).

Happy new year.

Fix failure to save screen notes when making a new one (github bug #6).

Add --config option to dcpomatic2 (#1703).

Add --version option to main DCP-o-matic.

Tweak certficate chain dialogue layout (#1530).

Another comment typo.

Fix error when trying to make DCP-o-matic DKDMs due to attempts to make
the validity time wider than that of the signing certificate.

Comment typo.

Remove our forcing of analyzeduration and probesize as they seem to
cause weird problems when scanning some files.  The unit test case
added here is a case in point: before this patch one
stream reported an incorrect channel count and sample rate.

It's a bit worrying as these parameters have been in DoM for many years,
but perhaps they have become unnecessary / harmful with changes to FFmpeg.

Add an output audio matrix (#1482).

Move sound output driver selection into new preferences tab.

Add an assertion.

Accept float fps when getting timecodes from the Timecode control
otherwise inaccurate values are returned for non-integer rates.
Should fix #1694.

Forward-ported from 1229de2042c785a9e851e5fc53922a75bfbf195d in master.