Fix warnings from adding CHECKED annotations

This centered on checking the result of sending IPC console heartbeats and a
few other one-offs.

src/IpcConsoleClient/Main.cpp

@@ -38,9 +38,16 @@ void heartbeat(std::atomic<bool> *running, werk::IpcConsoleClient *client)
 	delay.tv_sec = 1;
 	delay.tv_nsec = 0;
 
+	uint32_t failedHeartbeats = 0;
 	while (*running) {
 		nanosleep(&delay, nullptr);
-		client->heartbeat();
+		if (!client->heartbeat()) {
+			std::cout << "Failed to send heartbeat..." << std::endl;
+			failedHeartbeats += 1;
+			if (3 <= failedHeartbeats) {
+				break;
+			}
+		}
 	}
 }
 

src/Werk/Config/Config.cpp

@@ -56,17 +56,21 @@ void Config::execute()
 }
 
 //Run on whatever thread the configurables live on
-void Config::reloadConfigurables()
+bool Config::reloadConfigurables()
 {
 	//Don't reload unless changed
 	if (!_changed.value()) {
-		return;
+		return true;
 	}
 	_changed.reset();
 
+	bool success = true;
 	for (Configurable *configurable : _configurables) {
-		configurable->reloadConfig(*this);
+		if (!configurable->reloadConfig(*this)) {
+			success = false;
+		}
 	}
+	return success;
 }
 
 //Basic types

src/Werk/Config/Config.hpp

@@ -106,7 +106,7 @@ class Config : public Action
 	void execute() override;
 
 	//Run on whatever thread the configurables live on
-	void reloadConfigurables();
+	CHECKED bool reloadConfigurables();
 
 	//Basic value-as-string accessor - this is the method that every inheriting class must override (return nullptr if the key is missing)
 	CHECKED const char *getStringRaw(const std::string &key) const {

src/Werk/Console/IpcConsoleClient.hpp

@@ -43,7 +43,8 @@ class IpcConsoleClient : public NamedObject
 		_queue(boost::interprocess::open_only, name.c_str()),
 		_pid(getpid()) {
 		//Send a heartbeat to let the server know about the connection
-		heartbeat();
+		//The return value of this one is ignored
+		(void) heartbeat();
 	}
 
 	CHECKED bool send(const std::string &message);