Fix a couple of places where we could segfault while looking at a VF.

Report every frame (with index) that is over (or nearly over) the size limit (DoM #2698).

Report every frame (with index) that has a JPEG2000 codestream error (DoM #2698).

Fix operator== to take everything in VerificationNote into account.

Only write tool name into help, not its full path.

Fix bad indentation in verify tool help.

Bump libcxml for new feature.

Add some MCA IDs that claim to be from ClipsterDCI 5.9.3.5.

Re-read MXF descriptor after adding a key to a SMPTE subtitle asset (DoM #2660).

Fix checking for existing key_id; _key_id would always be set because MXF::set_key() sets it.

Allow use of KDMs when verifying DCPs.

Report progress with done/total rather than a float.

Simple pass-through of <Ruby> tags in subtitles.

Tolerate VI-N as a MCA channel ID - apparently written by CineAsset.

Cleanup: remove unnecessary initialization.

Cleanup: remove pointless explicit.

Cleanup: using sorting.

Cleanup: header sorting.

Cleanup: remove pointless explicit.

Remove logging of clamped values.

I can't remember seeing this logged for ages and it appears to cost
nearly a second on the rgb_xyz benchmark on AMD Ryzen 7 5700U.

Use an integer LUT for PiecewiseLUT2, hence removing a lrint and a multiply from the rgb -> xyz loop.

Support integer LUTs.

Rename lut -> double_lut.

Cleanup: remove out-of-date comment.

Cleanup: remove stray space.

Add rgb_to_xyz for packed int16_t.

Adapted from a patch from Aaron Boxer for grok support.

Split rgb_to_xyz() into two parts.

Don't write MCA subdescriptors for channels that don't exist in the MXF (#2631).

This causes some test ID churn.

Cleanup: two lines can be one.

Cleanup: add some comments.

Tolerate LsLss / RsRss as channel IDs (#2629).

I didn't yet find a reference which tells me what they are, but there
have been multiple reports of DCPs that have this channel ID and so
(before) could not be added to a DoM project.

Cleanup: id -> ID.

Fix argument parsing on Ubuntu 16.04.

Don't call BOOST_CHECK during static initialization.

The boost test library shipped with Ubuntu 16.04 throws an exception
if you do that.

Replace BOOST_TEST with an older BOOST_CHECK for Ubuntu 16.04.

Avoid full_name() so we can build on Ubuntu 16.04.

Avoid depends_on() so we can build on Ubuntu 16.04.

Switch to testing on Ubuntu 16.04 and 22.04.

Implement weakly_canonical for boost versions without it (e.g. the one on Ubuntu 16.04).

Add more filesystem wrappers that DoM needs.

Add wrappers around boost::filesystem methods that handle the
required mangling of long filenames on Windows.

Also wrap lots of missing places (e.g. calls to asdcplib, libxml++,
libcxml etc.) in dcp::filesystem::fix_long_path().

The idea is to keep paths un-mangled until they we call some
filesystem-related API and mangle them at that point.  Otherwise
we end up serialising mangled names, which seems like it will
not end well.

Should fix DoM #2623.

Cleanup: sorting.

Remove odd assertion stopping TZ minutes being more than 30 or less than -30 (DoM/GH #20).

Report failures to read resources from MXF files.

Bump the PNG subtitle / font buffer a little.

Fix build when there is no {Image,Graphics}Magick

Don't build dcpdumpimage unless we have {Image,Graphics}Magick

Cleanup: use a loop.

Add dcpdumpimage tool.

Check for missing font files referred from Interop closed captions (as well as subs).

Fix Windows build failure.

Accept any case for subtitle alignment specifiers (#2602).

SMPTE seems pretty clear to me that these should be lower case, but
counterexamples have been seen in the wild and it seems to make
sense to allow them.

Perhaps the verifier should raise an error with non-lower-case
examples.

Add some tests.

Cleanup: extract HAlign to its own files.

Cleanup: extract VAlign to its own files.

Cleanup: header ordering.

Bump asdcplib for probably-unimportant undefined variable use fix.

Fix use-after-free in error case.

Fix dcp::LocalTime constructor to cope with longer fractional second parts (DoM #2597).

Add check for empty <LabelText> in <ContentVersion>

We have a report of Deluxe failing a DCP because of this.

Cleanup: comment formatting for auto-generated docs.

Cleanup: make some default constructors more consistent.

Fix boost bind placeholder warnings.

Cleanup: header ordering.

Remove unused method.

Ignore some ASDCP warnings.

Add equality option sound_assets_can_differ.

Rationalise #undef-ing of ERROR.

Include trimming.

Cleanup: move EqualityOptions into its own file.

Cleanup: forward declaration sorting.

Cleanup: include sorting.

Cleanup: pass EqualityOptions as const&

This exposes a useless assignment of export_differing_subtitles,
which I've removed (and altered the comment accordingly).

Cleanup: use = default.

Make EqualityOptions into a class.

Cleanup: fix typo in comment.

Add asset_hashes_can_differ option to the equality checks.

Before recent changes Asset::_hash would be empty when the equality
checks were run (and Asset::equals() compares them directly).

This mean that differences in asset hash were being ignored, but
are no longer; so now we need this option to restore that behaviour
where required.

When reading a DCP set up asset hashes from the CPL/PKL, not by digesting the actual asset.

Previously when reading a DCP we would recalculate asset hashes.  This meant that corrupted
DCPs could be loaded in and their hashes re-written (if nobody checked the new hashes against
the ones in the PKL/CPL).

It seems better to take the hashes from the metadata.  Then if the assets are corrupted
and the DCP is re-written the PKL/CPL hashes will be preserved, showing rather than hiding
the corruption.

Add Asset::unset_hash().

Cleanup: remove unnecessary namespace.

Fix tests that were writing assets with the same filename to different reels.

Cleanup: sorting.

Move Editor class from verify_test.cc to test.cc

Add verifier check for the actual asset file's ID not being the same as the one in the asset map.

Cleanup: better variable names.

Cleanup: const.

Add Asset::set_file_preserving_hash().

Cleanup: set annotation text in make_simple().

Verify error if SMPTE subtitle asset has <Text> but no <LoadFont>.

Cleanup: replace a nasty try/catch.

Cleanup: formatting.

Cleanup: remove some unpleasant (and I think unnecessary) casting.

Rename MISSING_LOAD_FONT -> MISSING_LOAD_FONT_FOR_FONT.

Cleanup: comment typo fix.

Add SoundAsset::active_channels().

Fix stupid mistake introduced in 1e010ab5.

Don't unconditionally clear _fonts when inspecting assets (DoM #2536).

resolve_fonts() is sometimes called for the same asset with different
lists of things to check.  If it is called first with a font, and
then without one, we want to keep the font from the first call.

Cleanup: use find_if().

Cleanup: reduce some scopes.

More detailed error message.