Don't use C++ 17 features/syntax

Author: yhabteab

lib/base/dependencygraph.cpp

@@ -10,16 +10,18 @@ DependencyGraph::DependencyMap DependencyGraph::m_Dependencies;
 void DependencyGraph::AddDependency(ConfigObject* child, ConfigObject* parent)
 {
 	std::unique_lock<std::mutex> lock(m_Mutex);
-	if (auto [it, inserted] = m_Dependencies.insert(Edge(parent, child)); !inserted) {
-		m_Dependencies.modify(it, [](Edge& e) { e.count++; });
+	auto pair = m_Dependencies.insert(Edge(parent, child));
+	if (!pair.second) {
+		m_Dependencies.modify(pair.first, [](Edge& e) { e.count++; });
 	}
 }
 
 void DependencyGraph::RemoveDependency(ConfigObject* child, ConfigObject* parent)
 {
 	std::unique_lock<std::mutex> lock(m_Mutex);
 
-	if (auto it(m_Dependencies.find(Edge(parent, child))); it != m_Dependencies.end()) {
+	auto it(m_Dependencies.find(Edge(parent, child)));
+	if (it != m_Dependencies.end()) {
 		if (it->count > 1) {
 			// Remove a duplicate edge from child to node, i.e. decrement the corresponding counter.
 			m_Dependencies.modify(it, [](Edge& e) { e.count--; });

lib/remote/apilistener-configsync.cpp

@@ -501,7 +501,8 @@ void ApiListener::SendRuntimeConfigObjects(const JsonRpcConnection::Ptr& aclient
 
 	std::unordered_set<ConfigObject*> syncedObjects;
 	for (const Type::Ptr& type : Type::GetAllTypes()) {
-		if (auto *ctype = dynamic_cast<ConfigType *>(type.get())) {
+		auto *ctype = dynamic_cast<ConfigType *>(type.get());
+		if (ctype) {
 			for (const auto& object : ctype->GetObjects()) {
 				// All objects must be synced sorted by their dependency graph.
 				// Otherwise, downtimes/comments etc. might get synced before their respective Checkables, which will