Fix strange crash in verify().

This appears to be necessary to stop the first write to
state.subtitle_language from segfaulting.  It looks like the
optional<> is in some uninitialised / indeterminate state.

I could only reproduce this crash when running verify() from DoM,
not from the `dcpverify` tool.  And some simple test programs suggest
that even doing

State state;

should call the constructors of anything in State.

I don't think we need BOOST_NO_CXX11_SCOPED_ENUMS any more.

Cleanup: more test naming tweaks.

Tidy up some test names.

Cleanup: rename some check_ prefixes to verify_.

Cleanup: some more use of initialiser lists.

VerificationNote message tidying.

Tidy up parameters in VerificationNote.

Cleanup: now have using namespace boost::filesystem.

Fix tests to actually check the results in several cases.

_line should be optional.

Cleanup: remove lots of unnecessary namespace qualifiers.

Cleanup: tidy enum names for verification codes.

Cleanup: more use of prepare_directory().

Cleanup: more use of check_verify_result.

Cleanup: remove BRANCH file.

Bv2.1 10.1: if any asset is encrypted they all must be.

Rename encrypted() to any_encrypted() and add all_encrypted().

Bv2.1 9.2: PKL must be signed if it contains encrypted assets.

Bv2.1 9.1: PKL annotation text must match CPL ContentTitleText if there is only one CPL in the PKL.

Add some more access to PKL.

Bv2.1 8.7: CPLs with encrypted content must be signed.

Bv2.1 8.6.3: <ExtensionMetadata> must be present and have precise contents.

Bv2.1 8.6.{1.2}: CompositionMetadataAsset must exist and it must have some <VersionNumber>.

Add CPL::unset_version_number().

Bv2.1 8.5: FFOC and LFOC should be present and have particular values.

Require instrinsic duration of ReelMarkersAsset to be specified on construction.

We need the intrinsic duration to match other assets in the reel,
so we can't just calculate it from the markers.

Assert that entry_point and intrinsic_duration are not completely bogus.

Bv2.1 8.5: Features must have FFEC/FFMC markers.

Use feature not trailer for some tests to avoid verification errors about FFEC/FFMC.

Add CPL::set_content_kind.

Read scope attribute of ReleaseTerritory.

Check that the Editor actually edits something.

Use an enum class for Marker.

Bv2.1 8.4 We must have <Hash>es

Bv2.1 8.3.2: text tracks must have <EntryPoint> and it must be zero.

Add ReelAsset::unset_entry_point().

Bv2.1 8.3.1: MainSubtitles must be in all reels (if they are there at
all) and ClosedCaptions must have the same count on all reels.

Bv2.1 8.2: all parts of a reel must have the same duration.

Make subtitle checking robust to CPLs with no subs/ccaps in some reels.

Bv2.1 8.1: CPL <AnnotationText> should be the same as <ContentTitleText>

Bv2.1 8.1: CPL must have <AnnotationText>.

Make CPL annotation_text optional.

Fix some directory paths for tests.

Bv2.1 7.3: audio sample rate must be 48kHz.

Bv2.1 7.2.{6,7}: various limits on subtitle line and character counts.

Remove some unnecessary std:: qualifiers.

Bv2.1 7.2.{4,5}: check that subtitles are not too short, too close or
too early in the first reel.

Give simple_{picture,sound} a length parameter and fix crazy implementation of simple_sound().

Change subtitle SubtitleAsset API to return const Subtitle pointers; add subtitles_in_reel().

Replace std::list with std::vector in the API.

c++11 and other tidying.

BRANCH

Stop tests failing when libdcp version changes.

Bv2.1 7.2.4: first subtitle should be at least 4s into the DCP.

BRANCH

c++11 tidying.

Bv2.1 7.2.3: Check that subtitle <StartTime> exists and is 0.

Bv2.1 7.2.2: Check that subtitle languages are the same for all reels.

Remove unused variable.

Change the word 'disagree' to 'differ'.

Bv2.1 7.2.2: Check that subtitle Language tags are present.

Fix to allow re-writing of SMPTE subtitles.

Bv2.1 7.2.1: Check total size of all timed text fonts is not larger than 10MB.

I'm not sure if this is what Bv2.1 means, but hopefully it's close
enough.

Bv2.1 7.2.1: Check size of timed text asset is not larger than 115MB.

Update BRANCH.

Bv2.1 7.2.1: Check size of closed caption XML is not larger than 256KB.

Subtitle constructor should not be public.

Bv2.1 7.1: Check picture size and frame rate.

Extent black_image() to make images of any size.

Raise an exception to head off a weird asdcplib error with empty JPEG2000 comments.

Throw an exception on making a DCP with no reels.

Replace dcp::File with dcp::ArrayData.

Remove 32x32 test image.

Use a valid frame size for simple_picture().

Rename some notes to clarify places where size refers to bytes, not pixels.

Bv2.1 6.2.1: Check that release territory conforms to RFC 5646.

Bv2.1 6.2.1: Check that additional subtitle languages conform to RFC 5646.

Bv2.1 6.2.1: Check that closed caption reel and XML <Language> conform to RFC 5646.

Fix namespace on closed caption Language tag.

Small tweaks to how things are passed around.

Bv2.1 6.2.1: Check that the sound MXF Language tag conforms to RFC 5646.

Store sound asset language as a string, but set it as a
dcp::LanguageTag.

This makes it work the same as other language tags, and tolerates
bad values while trying not to allow them to be created.

Extract simple_sound() method.

We can always dump notes where there are none expected.

LanguageTag strings should not care about case.

 Bv2.1 6.2.1: Check that subtitle XML <Language> conforms to RFC 5646.

Tiny cleanup of using prepare_directory.

Bv2.1 6.2.1: Check that subtitle reel <Language> conforms to RFC 5646.

BRANCH

Bv2.1 6.1: DCPs must be SMPTE.

Add VERIFY_BV21_ERROR.

Add BRANCH notes.

Fix macOS test environment path.

Bump libcxml.

More c++11 tidying.

c++11 tidying.

Remove use of xmldiff in tests.

std::shared_ptr

Fix test following previous commit.