summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorCarl Hetherington <cth@carlh.net>2023-01-01 21:27:16 +0100
committerCarl Hetherington <cth@carlh.net>2023-01-07 22:46:32 +0100
commitcb574824a2e5d280b421e8c4e81fa90b85b877ae (patch)
tree15e91a4e95d18ad506e6ef5f591633581170ecf0 /src
parent05e0890d4bb991fc321ac2e11993eba37d20e077 (diff)
Cleanup: rename AssetMap::path -> file for consistency.
Diffstat (limited to 'src')
-rw-r--r--src/asset_map.cc16
-rw-r--r--src/asset_map.h6
-rw-r--r--src/dcp.h4
-rw-r--r--src/verify.cc8
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;
};
diff --git a/src/dcp.h b/src/dcp.h
index 941df67c..7d3d670a 100644
--- a/src/dcp.h
+++ b/src/dcp.h
@@ -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});