From 06d43e27c412083cf704af48ea40e5c589504240 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Christian=20Gl=C3=B6ckner?= <cgloeckner@freenet.de>
Date: Fri, 22 Jan 2016 09:39:50 +0100
Subject: stdcerr output overhaul

---
 drumgizmo/output/wavfile.cc | 22 ++++++++++++++++++----
 1 file changed, 18 insertions(+), 4 deletions(-)

(limited to 'drumgizmo/output/wavfile.cc')

diff --git a/drumgizmo/output/wavfile.cc b/drumgizmo/output/wavfile.cc
index ae8b3f1..d8b8d99 100644
--- a/drumgizmo/output/wavfile.cc
+++ b/drumgizmo/output/wavfile.cc
@@ -24,6 +24,8 @@
  *  along with DrumGizmo; if not, write to the Free Software
  *  Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA.
  */
+#include <iostream>
+
 #include "wavfile.h"
 
 WavfileOutputEngine::WavfileOutputEngine()
@@ -54,7 +56,8 @@ bool WavfileOutputEngine::init(Channels data) {
 		auto fname = file + data[i].name + "-" + std::to_string(i) + ".wav";
 		channels[i] = sf_open(fname.c_str(), SFM_WRITE, &info);
 		if (channels[i] == nullptr) {
-			printf("Write error...\n");
+			std::cerr << "[WaffileOutputEngine] Failed to initialize "
+				<< "channel #" << i << "\n";
 			return false;
 		}
 	}
@@ -63,11 +66,21 @@ bool WavfileOutputEngine::init(Channels data) {
 
 void WavfileOutputEngine::setParm(std::string parm, std::string value) {
 	if (parm == "file") {
+		// apply output filename prefix
 		file = value;
+		
 	} else if (parm == "srate") {
-		info.samplerate = std::stoi(value);
+		// try to apply samplerate
+		try {
+			info.samplerate = std::stoi(value);
+		} catch (...) {
+			std::cerr << "[WavfileOutputEngine] Invalid samplerate "
+				<< value << "\n";
+		}
+		
 	} else {
-		printf("Unsupported wavfile parameter '%s'\n", parm.c_str());
+		std::cerr << "[WavfileOutputEngine] Unsupported parameter '"
+			<< parm << "'\n";
 	}
 }
 
@@ -83,7 +96,8 @@ void WavfileOutputEngine::pre(size_t nsamples) {
 
 void WavfileOutputEngine::run(int ch, sample_t* samples, size_t nsamples) {
 	if (static_cast<unsigned int>(ch) >= channels.size()) {
-		printf("Invalid channel %d (%d channels available)", ch, static_cast<int>(channels.size()));
+		std::cerr << "[WavfileOutputEngine] cannot access channel #"
+			<< ch << " (" << channels.size() << " channels available)\n";
 		return;
 	}
 	
-- 
cgit v1.2.3