From 0a5615c17edf6d9c51d9269c824d6caf71f710e5 Mon Sep 17 00:00:00 2001 From: Carl Hetherington Date: Mon, 17 Jan 2022 18:45:19 +0100 Subject: Use optional for ReelAsset _annotation_text. Not only is this tag optional in Interop and SMPTE, but it would appear that if it is present but empty a DCP will not play back on Sony SRX320 / LMT3000 systems (DoM bug #2124). Here we use optional<>, as seems to make sense, and also refuse to write empty tags (instead omitting the tag entirely). --- test/ref/DCP/dcp_test1/ASSETMAP.xml | 2 +- .../cpl_81fb54df-e1bf-4647-8788-ea7ba154375b.xml | 3 --- .../pkl_6af1e0c1-c441-47f8-a502-3efd46b1fa4f.xml | 4 ++-- test/ref/DCP/dcp_test2/ASSETMAP.xml | 2 +- .../cpl_81fb54df-e1bf-4647-8788-ea7ba154375b.xml | 2 -- .../pkl_6af1e0c1-c441-47f8-a502-3efd46b1fa4f.xml | 4 ++-- test/ref/DCP/dcp_test5/ASSETMAP.xml | 2 +- .../cpl_81fb54df-e1bf-4647-8788-ea7ba154375b.xml | 3 --- .../pkl_5d51e8a1-b2a5-4da6-9b66-4615c3609440.xml | 4 ++-- test/ref/DCP/dcp_test7/ASSETMAP | 2 +- .../cpl_81fb54df-e1bf-4647-8788-ea7ba154375b.xml | 3 --- .../pkl_6af1e0c1-c441-47f8-a502-3efd46b1fa4f.xml | 4 ++-- test/ref/DCP/encryption_test/ASSETMAP.xml | 2 +- .../cpl_81fb54df-e1bf-4647-8788-ea7ba154375b.xml | 16 +++++++--------- .../pkl_8ee8f7da-8da2-4adb-ae0e-31e8f4b91900.xml | 18 +++++++++--------- test/ref/cpl_metadata_test1.xml | 1 - test/ref/cpl_metadata_test2.xml | 1 - test/ref/write_interop_subtitle_test3/ASSETMAP | 2 +- .../pkl_e94b8a0d-27f7-408a-af16-78d3df419a91.xml | 4 ++-- 19 files changed, 32 insertions(+), 47 deletions(-) (limited to 'test/ref') diff --git a/test/ref/DCP/dcp_test1/ASSETMAP.xml b/test/ref/DCP/dcp_test1/ASSETMAP.xml index 10083feb..ee780c7c 100644 --- a/test/ref/DCP/dcp_test1/ASSETMAP.xml +++ b/test/ref/DCP/dcp_test1/ASSETMAP.xml @@ -26,7 +26,7 @@ cpl_81fb54df-e1bf-4647-8788-ea7ba154375b.xml 1 0 - 8762 + 8630 diff --git a/test/ref/DCP/dcp_test1/cpl_81fb54df-e1bf-4647-8788-ea7ba154375b.xml b/test/ref/DCP/dcp_test1/cpl_81fb54df-e1bf-4647-8788-ea7ba154375b.xml index ae0f23be..01eea88d 100644 --- a/test/ref/DCP/dcp_test1/cpl_81fb54df-e1bf-4647-8788-ea7ba154375b.xml +++ b/test/ref/DCP/dcp_test1/cpl_81fb54df-e1bf-4647-8788-ea7ba154375b.xml @@ -18,7 +18,6 @@ urn:uuid:93182bd2-b1e8-41a3-b5c8-6e6564273bff - 24 1 24 0 @@ -36,7 +35,6 @@ urn:uuid:46c3eb45-15e5-47d6-8684-d8641e4dc516 - 24 1 24 0 @@ -47,7 +45,6 @@ urn:uuid:8b92bcee-62fc-4a33-a51a-816e9611ce85 - 24 1 24 0 diff --git a/test/ref/DCP/dcp_test1/pkl_6af1e0c1-c441-47f8-a502-3efd46b1fa4f.xml b/test/ref/DCP/dcp_test1/pkl_6af1e0c1-c441-47f8-a502-3efd46b1fa4f.xml index 3f285d23..cf753574 100644 --- a/test/ref/DCP/dcp_test1/pkl_6af1e0c1-c441-47f8-a502-3efd46b1fa4f.xml +++ b/test/ref/DCP/dcp_test1/pkl_6af1e0c1-c441-47f8-a502-3efd46b1fa4f.xml @@ -9,8 +9,8 @@ urn:uuid:81fb54df-e1bf-4647-8788-ea7ba154375b 81fb54df-e1bf-4647-8788-ea7ba154375b - c1DRq6GaSzV2brF0YnSNed46nqk= - 8762 + 6pkiSEIBuZW7KEY73GrNNw8UjDE= + 8630 text/xml diff --git a/test/ref/DCP/dcp_test2/ASSETMAP.xml b/test/ref/DCP/dcp_test2/ASSETMAP.xml index 61cf5d21..23390415 100644 --- a/test/ref/DCP/dcp_test2/ASSETMAP.xml +++ b/test/ref/DCP/dcp_test2/ASSETMAP.xml @@ -26,7 +26,7 @@ cpl_81fb54df-e1bf-4647-8788-ea7ba154375b.xml 1 0 - 1774 + 1686 diff --git a/test/ref/DCP/dcp_test2/cpl_81fb54df-e1bf-4647-8788-ea7ba154375b.xml b/test/ref/DCP/dcp_test2/cpl_81fb54df-e1bf-4647-8788-ea7ba154375b.xml index 6e5fab73..3910dcd3 100644 --- a/test/ref/DCP/dcp_test2/cpl_81fb54df-e1bf-4647-8788-ea7ba154375b.xml +++ b/test/ref/DCP/dcp_test2/cpl_81fb54df-e1bf-4647-8788-ea7ba154375b.xml @@ -18,7 +18,6 @@ urn:uuid:77e1fb48-ce0c-4d29-bf88-8c3bfec8013a - 24 1 24 0 @@ -27,7 +26,6 @@ urn:uuid:46c3eb45-15e5-47d6-8684-d8641e4dc516 - 24 1 24 0 diff --git a/test/ref/DCP/dcp_test2/pkl_6af1e0c1-c441-47f8-a502-3efd46b1fa4f.xml b/test/ref/DCP/dcp_test2/pkl_6af1e0c1-c441-47f8-a502-3efd46b1fa4f.xml index a37a062a..abfd5100 100644 --- a/test/ref/DCP/dcp_test2/pkl_6af1e0c1-c441-47f8-a502-3efd46b1fa4f.xml +++ b/test/ref/DCP/dcp_test2/pkl_6af1e0c1-c441-47f8-a502-3efd46b1fa4f.xml @@ -9,8 +9,8 @@ urn:uuid:81fb54df-e1bf-4647-8788-ea7ba154375b 81fb54df-e1bf-4647-8788-ea7ba154375b - Q9GRM2MtKZAg5FEIEkRsL28rNeQ= - 1774 + 9JCFUMDcUujOZxSqSig3u/5wIuI= + 1686 text/xml diff --git a/test/ref/DCP/dcp_test5/ASSETMAP.xml b/test/ref/DCP/dcp_test5/ASSETMAP.xml index 3f5d9076..fb9f897a 100644 --- a/test/ref/DCP/dcp_test5/ASSETMAP.xml +++ b/test/ref/DCP/dcp_test5/ASSETMAP.xml @@ -26,7 +26,7 @@ cpl_81fb54df-e1bf-4647-8788-ea7ba154375b.xml 1 0 - 2156 + 2024 diff --git a/test/ref/DCP/dcp_test5/cpl_81fb54df-e1bf-4647-8788-ea7ba154375b.xml b/test/ref/DCP/dcp_test5/cpl_81fb54df-e1bf-4647-8788-ea7ba154375b.xml index a7a1441e..69558723 100644 --- a/test/ref/DCP/dcp_test5/cpl_81fb54df-e1bf-4647-8788-ea7ba154375b.xml +++ b/test/ref/DCP/dcp_test5/cpl_81fb54df-e1bf-4647-8788-ea7ba154375b.xml @@ -18,7 +18,6 @@ urn:uuid:46c3eb45-15e5-47d6-8684-d8641e4dc516 - 24 1 24 0 @@ -29,7 +28,6 @@ urn:uuid:8b92bcee-62fc-4a33-a51a-816e9611ce85 - 24 1 24 0 @@ -38,7 +36,6 @@ urn:uuid:b68febcc-5ddf-489a-84a7-924f29fa2afd - 24 1 1489 0 diff --git a/test/ref/DCP/dcp_test5/pkl_5d51e8a1-b2a5-4da6-9b66-4615c3609440.xml b/test/ref/DCP/dcp_test5/pkl_5d51e8a1-b2a5-4da6-9b66-4615c3609440.xml index 6f7974a4..0ae84d3d 100644 --- a/test/ref/DCP/dcp_test5/pkl_5d51e8a1-b2a5-4da6-9b66-4615c3609440.xml +++ b/test/ref/DCP/dcp_test5/pkl_5d51e8a1-b2a5-4da6-9b66-4615c3609440.xml @@ -9,8 +9,8 @@ urn:uuid:81fb54df-e1bf-4647-8788-ea7ba154375b 81fb54df-e1bf-4647-8788-ea7ba154375b - kvvGY8+ZNsyJyWk6i5Ha1dRpFQU= - 2156 + +a4c/A2gZJW4rVH5BZ7jbb6qXkc= + 2024 text/xml diff --git a/test/ref/DCP/dcp_test7/ASSETMAP b/test/ref/DCP/dcp_test7/ASSETMAP index f82ce56f..71873d99 100644 --- a/test/ref/DCP/dcp_test7/ASSETMAP +++ b/test/ref/DCP/dcp_test7/ASSETMAP @@ -26,7 +26,7 @@ cpl_81fb54df-e1bf-4647-8788-ea7ba154375b.xml 1 0 - 2168 + 2036 diff --git a/test/ref/DCP/dcp_test7/cpl_81fb54df-e1bf-4647-8788-ea7ba154375b.xml b/test/ref/DCP/dcp_test7/cpl_81fb54df-e1bf-4647-8788-ea7ba154375b.xml index d3589fc5..89f6851d 100644 --- a/test/ref/DCP/dcp_test7/cpl_81fb54df-e1bf-4647-8788-ea7ba154375b.xml +++ b/test/ref/DCP/dcp_test7/cpl_81fb54df-e1bf-4647-8788-ea7ba154375b.xml @@ -18,7 +18,6 @@ urn:uuid:93182bd2-b1e8-41a3-b5c8-6e6564273bff - 24 1 24 0 @@ -36,7 +35,6 @@ urn:uuid:46c3eb45-15e5-47d6-8684-d8641e4dc516 - 24 1 24 0 @@ -47,7 +45,6 @@ urn:uuid:8b92bcee-62fc-4a33-a51a-816e9611ce85 - 24 1 24 0 diff --git a/test/ref/DCP/dcp_test7/pkl_6af1e0c1-c441-47f8-a502-3efd46b1fa4f.xml b/test/ref/DCP/dcp_test7/pkl_6af1e0c1-c441-47f8-a502-3efd46b1fa4f.xml index a2c83a0e..6d529422 100644 --- a/test/ref/DCP/dcp_test7/pkl_6af1e0c1-c441-47f8-a502-3efd46b1fa4f.xml +++ b/test/ref/DCP/dcp_test7/pkl_6af1e0c1-c441-47f8-a502-3efd46b1fa4f.xml @@ -9,8 +9,8 @@ urn:uuid:81fb54df-e1bf-4647-8788-ea7ba154375b 81fb54df-e1bf-4647-8788-ea7ba154375b - LE8AZqXvZYb8W6QjVZiN7zB8Tro= - 2168 + qD31AlG29UXWwTGYch5P+zuD3xc= + 2036 text/xml;asdcpKind=CPL diff --git a/test/ref/DCP/encryption_test/ASSETMAP.xml b/test/ref/DCP/encryption_test/ASSETMAP.xml index 3c1ca16c..71fa54ac 100644 --- a/test/ref/DCP/encryption_test/ASSETMAP.xml +++ b/test/ref/DCP/encryption_test/ASSETMAP.xml @@ -26,7 +26,7 @@ cpl_81fb54df-e1bf-4647-8788-ea7ba154375b.xml 1 0 - 9314 + 9226 diff --git a/test/ref/DCP/encryption_test/cpl_81fb54df-e1bf-4647-8788-ea7ba154375b.xml b/test/ref/DCP/encryption_test/cpl_81fb54df-e1bf-4647-8788-ea7ba154375b.xml index 598444b4..2866df6d 100644 --- a/test/ref/DCP/encryption_test/cpl_81fb54df-e1bf-4647-8788-ea7ba154375b.xml +++ b/test/ref/DCP/encryption_test/cpl_81fb54df-e1bf-4647-8788-ea7ba154375b.xml @@ -18,7 +18,6 @@ urn:uuid:e98d059d-645f-4343-a30f-edc61d58b8e0 - 24 1 24 0 @@ -30,7 +29,6 @@ urn:uuid:11471a77-7cd8-4de6-8c11-beaa198c015f - 24 1 24 0 @@ -59,15 +57,15 @@ - G/4l/FrHjANRjHMvx1wwtojaqXY= + 4/8QgoO0blTh31eQ8ubu1zkECYc= - tLWdo5wkG1aseboTfSONMMYLEnd89Wqcjw0Gwhw71wX/lEKEGYiCluUchInurc6J -+Lq7rgNz+SPg7XIgh1N58EHqEarwH6oFTf2ZFMtoEzmX1nXcjKR7MRYomMtCBb/V -g+LOpPuOY+36bQ/8XBYiH8t1w6ayNSLI0LGTuAfJDz+/L6wAGRSLosQgVThjz3bE -nnl9V96ufU3vMPcrq2YpT99Rb61gX3ItBKW60FiTX+NB5vBwQ1lq4vJ6qno4aviE -HEebN6byReZtqm+X6/olVvVHJBAyOG1ZKOb9Oj6I2Iw4hx0OYB70Tg7Qkx+f8zsl -HHXyc9gA+KzKStrrm390oA== + jHGd0G9X7iT26LNertFIPsKGIVUxUNEZWSDmjzYH4QwP2SAYOMh/tGXAaUQyy2vA ++qCj+OQavpBS8q1HMekyUrShRo18XY6mpS3lSADdZBY7TLExCdkLtSq7yoPVfzx+ ++iRypVIKBl6xbv9wf/ssCzpxXMbsuaL5BGbOdRlgH7FFdQN5GKbAWVYoCmPHcl+q +S2BLqXN1S9LjB4zQ606bVq2C4S7mUYqAFZ0a9w/MK1RU5kFRJfq7W2ZktvhIWrFI +K6iF4XGhwoCURfPKP9piAwrpR+5sI7LuknMLa/JHPYzkBCH63oZoNW8vHyzCBNpR +ADgG10jZGAXp+GY0BL8v2w== diff --git a/test/ref/DCP/encryption_test/pkl_8ee8f7da-8da2-4adb-ae0e-31e8f4b91900.xml b/test/ref/DCP/encryption_test/pkl_8ee8f7da-8da2-4adb-ae0e-31e8f4b91900.xml index ecacaac5..b97d6eaf 100644 --- a/test/ref/DCP/encryption_test/pkl_8ee8f7da-8da2-4adb-ae0e-31e8f4b91900.xml +++ b/test/ref/DCP/encryption_test/pkl_8ee8f7da-8da2-4adb-ae0e-31e8f4b91900.xml @@ -9,8 +9,8 @@ urn:uuid:81fb54df-e1bf-4647-8788-ea7ba154375b 81fb54df-e1bf-4647-8788-ea7ba154375b - iBE1dh+gKFCi7R93mx2AoJ1eaek= - 9314 + zk4L2kXj9llDdjmxCW2d2X06VQM= + 9226 text/xml @@ -46,15 +46,15 @@ - iFEJZ3tiVaiuJMzVY/ju8qx+KV0= + xzO3cFwUX87TL4rL9jq6CLqDN6I= - GKFnAkzVuhXi391uAz5kRTmKaJKMKd3iAe1XtYsUoRfhVxtQCv/hha/W8xpVsP2J -g/duTyASL/T4vdaQsrR8K3JUwP/r/gWlN3UfcFIORHkKeJ9OOBSIdauthODZpAJw -V+ZfgTu5S8aTh4XqkWWeVDQSsBWJhgHWFM4vrPrsWxae4l3SGj2DpcXUPhLOe55+ -Xh/sVJVH8CpEcNGKNDZnaJShUI6Ige45ElUePZNsYRNLBDWDcYw+nR0DAUovEuCh -CFqB2H9FKwufxAIWqbEYSca4liHlgFYL7TuHgaDbzdjTlQ8oL3kCPGbtVX5oWiEu -ZksAYCwT9sDxSxeThmFZOQ== + VIZyD5JmV2gnPyzMEMwDaIg+NBOZzpSvBoRtPbWrYUyjxVzQfPvFzAAE+4hz8Wc7 +TIpiTZ3sYXKRQAEmA4Iu5AOJ2uijv8tnioo2FyV6JT9aGr5cNi7eeGFmN7ILU4zc +rgp99iWgTRms8T8C7zy/NtE8P2q8K4sxQvxnYmK/0kHkQPQnWpZafAd6G/fpoV0h +5W8UQTtn72OEPhm21uxp4crWRk9YlF2ayGYDBGm8YFP01w663e0r61xq6ROW2O4i +jR2dQWPM1nzrafHSJKg2TL4S/GtTsDNNuIJihzGpmEjULX2FIt83f6dogyr7K624 +XcjpahE+WHn7WR3VVZsMkw== diff --git a/test/ref/cpl_metadata_test1.xml b/test/ref/cpl_metadata_test1.xml index 58649e8d..1140c07d 100644 --- a/test/ref/cpl_metadata_test1.xml +++ b/test/ref/cpl_metadata_test1.xml @@ -18,7 +18,6 @@ urn:uuid:e98d059d-645f-4343-a30f-edc61d58b8e0 - 24 1 24 0 diff --git a/test/ref/cpl_metadata_test2.xml b/test/ref/cpl_metadata_test2.xml index 01dd52d8..d04d25af 100644 --- a/test/ref/cpl_metadata_test2.xml +++ b/test/ref/cpl_metadata_test2.xml @@ -18,7 +18,6 @@ urn:uuid:e98d059d-645f-4343-a30f-edc61d58b8e0 - 24 1 24 0 diff --git a/test/ref/write_interop_subtitle_test3/ASSETMAP b/test/ref/write_interop_subtitle_test3/ASSETMAP index 228a7e1c..7c901513 100644 --- a/test/ref/write_interop_subtitle_test3/ASSETMAP +++ b/test/ref/write_interop_subtitle_test3/ASSETMAP @@ -26,7 +26,7 @@ cpl_46c3eb45-15e5-47d6-8684-d8641e4dc516.xml 1 0 - 1105 + 1061 diff --git a/test/ref/write_interop_subtitle_test3/pkl_e94b8a0d-27f7-408a-af16-78d3df419a91.xml b/test/ref/write_interop_subtitle_test3/pkl_e94b8a0d-27f7-408a-af16-78d3df419a91.xml index 664ad626..2da8e910 100644 --- a/test/ref/write_interop_subtitle_test3/pkl_e94b8a0d-27f7-408a-af16-78d3df419a91.xml +++ b/test/ref/write_interop_subtitle_test3/pkl_e94b8a0d-27f7-408a-af16-78d3df419a91.xml @@ -9,8 +9,8 @@ urn:uuid:46c3eb45-15e5-47d6-8684-d8641e4dc516 46c3eb45-15e5-47d6-8684-d8641e4dc516 - YkdOdaGok/Vrkq62zAkfCw1l/Y4= - 1105 + 614bJ4VLsNZ6mLbdXbZXjGuoSsY= + 1061 text/xml;asdcpKind=CPL -- cgit v1.2.3