diff options
| author | Carl Hetherington <cth@carlh.net> | 2023-01-01 21:27:16 +0100 |
|---|---|---|
| committer | Carl Hetherington <cth@carlh.net> | 2023-01-07 22:46:32 +0100 |
| commit | cb574824a2e5d280b421e8c4e81fa90b85b877ae (patch) | |
| tree | 15e91a4e95d18ad506e6ef5f591633581170ecf0 /src | |
| parent | 05e0890d4bb991fc321ac2e11993eba37d20e077 (diff) | |
Cleanup: rename AssetMap::path -> file for consistency.
Diffstat (limited to 'src')
| -rw-r--r-- | src/asset_map.cc | 16 | ||||
| -rw-r--r-- | src/asset_map.h | 6 | ||||
| -rw-r--r-- | src/dcp.h | 4 | ||||
| -rw-r--r-- | src/verify.cc | 8 |
4 files changed, 17 insertions, 17 deletions
diff --git a/src/asset_map.cc b/src/asset_map.cc index 4b6d828c..c1bafd92 100644 --- a/src/asset_map.cc +++ b/src/asset_map.cc @@ -55,12 +55,12 @@ static string const assetmap_interop_ns = "http://www.digicine.com/PROTO-ASDCP-A static string const assetmap_smpte_ns = "http://www.smpte-ra.org/schemas/429-9/2007/AM"; -AssetMap::AssetMap(boost::filesystem::path path) - : _path(path) +AssetMap::AssetMap(boost::filesystem::path file) + : _file(file) { cxml::Document doc("AssetMap"); - doc.read_file(path); + doc.read_file(file); if (doc.namespace_uri() == assetmap_interop_ns) { _standard = Standard::INTEROP; } else if (doc.namespace_uri() == assetmap_smpte_ns) { @@ -76,7 +76,7 @@ AssetMap::AssetMap(boost::filesystem::path path) _creator = doc.string_child("Creator"); for (auto asset: doc.node_child("AssetList")->node_children("Asset")) { - _assets.push_back(Asset(asset, _path->parent_path(), _standard)); + _assets.push_back(Asset(asset, _file->parent_path(), _standard)); } } @@ -148,7 +148,7 @@ AssetMap::pkl_paths() const void -AssetMap::write_xml(boost::filesystem::path path) const +AssetMap::write_xml(boost::filesystem::path file) const { xmlpp::Document doc; xmlpp::Element* root; @@ -188,11 +188,11 @@ AssetMap::write_xml(boost::filesystem::path path) const auto asset_list = root->add_child("AssetList"); for (auto const& asset: _assets) { - asset.write_xml(asset_list, path.parent_path()); + asset.write_xml(asset_list, file.parent_path()); } - doc.write_to_file_formatted(path.string(), "UTF-8"); - _path = path; + doc.write_to_file_formatted(file.string(), "UTF-8"); + _file = file; } diff --git a/src/asset_map.h b/src/asset_map.h index 5238b96b..dcd0f961 100644 --- a/src/asset_map.h +++ b/src/asset_map.h @@ -54,8 +54,8 @@ public: explicit AssetMap(boost::filesystem::path path); - boost::optional<boost::filesystem::path> path() const { - return _path; + boost::optional<boost::filesystem::path> file() const { + return _file; } std::map<std::string, boost::filesystem::path> asset_ids_and_paths() const; @@ -99,7 +99,7 @@ public: private: std::vector<Asset> _assets; - mutable boost::optional<boost::filesystem::path> _path; + mutable boost::optional<boost::filesystem::path> _file; }; @@ -182,12 +182,12 @@ public: return _pkls; } - boost::optional<boost::filesystem::path> asset_map_path() const { + boost::optional<boost::filesystem::path> asset_map_file() const { if (!_asset_map) { return {}; } - return _asset_map->path(); + return _asset_map->file(); } boost::optional<AssetMap> asset_map() const { diff --git a/src/verify.cc b/src/verify.cc index c8009741..bfa697da 100644 --- a/src/verify.cc +++ b/src/verify.cc @@ -1649,12 +1649,12 @@ verify_assetmap( auto asset_map = dcp->asset_map(); DCP_ASSERT(asset_map); - validate_xml(asset_map->path().get(), xsd_dtd_directory, notes); + validate_xml(asset_map->file().get(), xsd_dtd_directory, notes); set<string> uuid_set; for (auto const& asset: asset_map->assets()) { if (!uuid_set.insert(asset.id()).second) { - notes.push_back({VerificationNote::Type::ERROR, VerificationNote::Code::DUPLICATE_ASSET_ID_IN_ASSETMAP, asset_map->id(), asset_map->path().get()}); + notes.push_back({VerificationNote::Type::ERROR, VerificationNote::Code::DUPLICATE_ASSET_ID_IN_ASSETMAP, asset_map->id(), asset_map->file().get()}); break; } } @@ -1725,8 +1725,8 @@ dcp::verify ( verify_pkl(dcp, pkl, *xsd_dtd_directory, notes); } - if (dcp->asset_map_path()) { - stage ("Checking ASSETMAP", dcp->asset_map_path().get()); + if (dcp->asset_map_file()) { + stage("Checking ASSETMAP", dcp->asset_map_file().get()); verify_assetmap(dcp, *xsd_dtd_directory, notes); } else { notes.push_back ({VerificationNote::Type::ERROR, VerificationNote::Code::MISSING_ASSETMAP}); |
