X-Git-Url: https://git.carlh.net/gitweb/?a=blobdiff_plain;f=src%2Flib%2Fcolour_conversion.cc;h=57e73a5b5c02133b65d30b5d544cc48a7b03a130;hb=444809fb888ed99803f2d19c94d3faef067cf348;hp=2e052060e2b247ad9f26dbd51c54399b952f9c91;hpb=c403e757cf0b029954fe18dc969314bfb179412f;p=dcpomatic.git

diff --git a/src/lib/colour_conversion.cc b/src/lib/colour_conversion.cc
index 2e052060e..57e73a5b5 100644
--- a/src/lib/colour_conversion.cc
+++ b/src/lib/colour_conversion.cc
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2013-2020 Carl Hetherington <cth@carlh.net>
+    Copyright (C) 2013-2021 Carl Hetherington <cth@carlh.net>
 
     This file is part of DCP-o-matic.
 
@@ -22,6 +22,7 @@
 #include "colour_conversion.h"
 #include "util.h"
 #include "digester.h"
+#include "warnings.h"
 #include <dcp/raw_convert.h>
 #include <dcp/chromaticity.h>
 #include <dcp/gamma_transfer_function.h>
@@ -29,8 +30,9 @@
 #include <dcp/identity_transfer_function.h>
 #include <dcp/s_gamut3_transfer_function.h>
 #include <libcxml/cxml.h>
+DCPOMATIC_DISABLE_WARNINGS
 #include <libxml++/libxml++.h>
-#include <boost/foreach.hpp>
+DCPOMATIC_ENABLE_WARNINGS
 #include <iostream>
 
 #include "i18n.h"
@@ -39,9 +41,10 @@ using std::list;
 using std::string;
 using std::cout;
 using std::vector;
-using boost::shared_ptr;
+using std::make_shared;
+using std::shared_ptr;
 using boost::optional;
-using boost::dynamic_pointer_cast;
+using std::dynamic_pointer_cast;
 using dcp::raw_convert;
 
 vector<PresetColourConversion> PresetColourConversion::_presets;
@@ -66,19 +69,19 @@ ColourConversion::ColourConversion (cxml::NodePtr node, int version)
 
 		/* Version 2.x */
 
-		cxml::ConstNodePtr in_node = node->node_child ("InputTransferFunction");
-		string in_type = in_node->string_child ("Type");
+		auto in_node = node->node_child ("InputTransferFunction");
+		auto in_type = in_node->string_child ("Type");
 		if (in_type == "Gamma") {
-			_in.reset (new dcp::GammaTransferFunction (in_node->number_child<double> ("Gamma")));
+			_in = make_shared<dcp::GammaTransferFunction>(in_node->number_child<double> ("Gamma"));
 		} else if (in_type == "ModifiedGamma") {
-			_in.reset (new dcp::ModifiedGammaTransferFunction (
-					   in_node->number_child<double> ("Power"),
-					   in_node->number_child<double> ("Threshold"),
-					   in_node->number_child<double> ("A"),
-					   in_node->number_child<double> ("B")
-					   ));
+			_in = make_shared<dcp::ModifiedGammaTransferFunction>(
+					   in_node->number_child<double>("Power"),
+					   in_node->number_child<double>("Threshold"),
+					   in_node->number_child<double>("A"),
+					   in_node->number_child<double>("B")
+					   );
 		} else if (in_type == "SGamut3") {
-			_in.reset (new dcp::SGamut3TransferFunction ());
+			_in = make_shared<dcp::SGamut3TransferFunction>();
 		}
 
 	} else {
@@ -94,14 +97,14 @@ ColourConversion::ColourConversion (cxml::NodePtr node, int version)
 
 	_yuv_to_rgb = static_cast<dcp::YUVToRGB> (node->optional_number_child<int>("YUVToRGB").get_value_or (dcp::YUV_TO_RGB_REC601));
 
-	list<cxml::NodePtr> m = node->node_children ("Matrix");
+	auto m = node->node_children ("Matrix");
 	if (!m.empty ()) {
 		/* Read in old <Matrix> nodes and convert them to chromaticities */
 		boost::numeric::ublas::matrix<double> C (3, 3);
-		for (list<cxml::NodePtr>::iterator i = m.begin(); i != m.end(); ++i) {
-			int const ti = (*i)->number_attribute<int> ("i");
-			int const tj = (*i)->number_attribute<int> ("j");
-			C(ti, tj) = raw_convert<double> ((*i)->content ());
+		for (auto i: m) {
+			int const ti = i->number_attribute<int>("i");
+			int const tj = i->number_attribute<int>("j");
+			C(ti, tj) = raw_convert<double>(i->content());
 		}
 
 		double const rd = C(0, 0) + C(1, 0) + C(2, 0);
@@ -125,11 +128,11 @@ ColourConversion::ColourConversion (cxml::NodePtr node, int version)
 		}
 	}
 
-	optional<double> gamma = node->optional_number_child<double> ("OutputGamma");
+	auto gamma = node->optional_number_child<double>("OutputGamma");
 	if (gamma) {
-		_out.reset (new dcp::GammaTransferFunction (node->number_child<double> ("OutputGamma")));
+		_out = make_shared<dcp::GammaTransferFunction>(node->number_child<double>("OutputGamma"));
 	} else {
-		_out.reset (new dcp::IdentityTransferFunction ());
+		_out = make_shared<dcp::IdentityTransferFunction>();
 	}
 }
 
@@ -146,13 +149,13 @@ ColourConversion::from_xml (cxml::NodePtr node, int version)
 void
 ColourConversion::as_xml (xmlpp::Node* node) const
 {
-	xmlpp::Node* in_node = node->add_child ("InputTransferFunction");
+	auto in_node = node->add_child ("InputTransferFunction");
 	if (dynamic_pointer_cast<const dcp::GammaTransferFunction> (_in)) {
-		shared_ptr<const dcp::GammaTransferFunction> tf = dynamic_pointer_cast<const dcp::GammaTransferFunction> (_in);
+		auto tf = dynamic_pointer_cast<const dcp::GammaTransferFunction> (_in);
 		in_node->add_child("Type")->add_child_text ("Gamma");
 		in_node->add_child("Gamma")->add_child_text (raw_convert<string> (tf->gamma ()));
 	} else if (dynamic_pointer_cast<const dcp::ModifiedGammaTransferFunction> (_in)) {
-		shared_ptr<const dcp::ModifiedGammaTransferFunction> tf = dynamic_pointer_cast<const dcp::ModifiedGammaTransferFunction> (_in);
+		auto tf = dynamic_pointer_cast<const dcp::ModifiedGammaTransferFunction> (_in);
 		in_node->add_child("Type")->add_child_text ("ModifiedGamma");
 		in_node->add_child("Power")->add_child_text (raw_convert<string> (tf->power ()));
 		in_node->add_child("Threshold")->add_child_text (raw_convert<string> (tf->threshold ()));
@@ -185,14 +188,14 @@ ColourConversion::as_xml (xmlpp::Node* node) const
 optional<size_t>
 ColourConversion::preset () const
 {
-	vector<PresetColourConversion> presets = PresetColourConversion::all ();
+	auto presets = PresetColourConversion::all ();
 	size_t i = 0;
 	while (i < presets.size() && presets[i].conversion != *this) {
 		++i;
 	}
 
 	if (i >= presets.size ()) {
-		return optional<size_t> ();
+		return {};
 	}
 
 	return i;
@@ -228,7 +231,7 @@ ColourConversion::identifier () const
 		digester.add (_adjusted_white.get().y);
 	}
 
-	shared_ptr<const dcp::GammaTransferFunction> gf = dynamic_pointer_cast<const dcp::GammaTransferFunction> (_out);
+	auto gf = dynamic_pointer_cast<const dcp::GammaTransferFunction> (_out);
 	if (gf) {
 		digester.add (gf->gamma ());
 	}
@@ -290,7 +293,7 @@ PresetColourConversion::setup_colour_conversion_presets ()
 PresetColourConversion
 PresetColourConversion::from_id (string s)
 {
-	BOOST_FOREACH (PresetColourConversion const& i, _presets) {
+	for (auto const& i: _presets) {
 		if (i.id == s) {
 			return i;
 		}