Merge pull request #54 from timohl/master

Added more void casts to prevent unused warnings.

Author: timohl

src/message/migfra/task.cpp

@@ -176,7 +176,7 @@ void Task_container::load(const YAML::Node &node)
 	std::string type;
 	try {
 		fast::load(type, node["task"]);
-	} catch (const std::exception /*&e*/) {
+	} catch (const std::exception &/*e*/) {
 		throw Task_container::no_task_exception("Cannot find key \"task\" to load Task from YAML.");
 	}
 	if (type == "start vm") {

src/mqtt_communicator.cpp

@@ -328,7 +328,7 @@ std::string MQTT_communicator::get_message(const std::string &topic, const std::
 		auto &subscription = subscriptions.at(topic);
 		lock.unlock();
 		return subscription->get_message(duration, actual_topic);
-	} catch (const std::out_of_range /*&e*/) {
+	} catch (const std::out_of_range &/*e*/) {
 		throw std::out_of_range("Topic not found in subscriptions.");
 	}
 }

test/optional_test.cpp

@@ -58,15 +58,15 @@ struct Task_tester :
 		fructose_assert_eq(tag, p.get_tag());
 		fructose_assert_exception(p.get(), std::runtime_error);
 		fructose_assert_exception(*p, std::runtime_error);
-		fructose_assert_exception(p->id, std::runtime_error);
+		fructose_assert_exception((void) p->id, std::runtime_error);
 		p = Person(1);
 		fructose_assert(p);
 		fructose_assert(p.is_valid());
 		fructose_assert_no_exception(p.get());
 		fructose_assert_no_exception(*p);
-		fructose_assert_no_exception((*p).id);
-		fructose_assert_no_exception(p.get().id);
-		fructose_assert_no_exception(p->id);
+		fructose_assert_no_exception((void) (*p).id);
+		fructose_assert_no_exception((void) p.get().id);
+		fructose_assert_no_exception((void) p->id);
 		fructose_assert((*p).id == 1);
 		fructose_assert(p.get().id == 1);
 		fructose_assert(p->id == 1);