code quality: consolidated helpers to single header

main.cpp

@@ -3,10 +3,10 @@
 #include <iostream>
 #include <sstream>
 
-#include "qrcodegen.hpp"
-#include "QR.hpp"
+#include <qrcodegen.hpp>
 
-#define UNUSED(var) (void) var;
+#include "util.hpp"
+#include "QR.hpp"
 
 // Version is injected through cmake
 constexpr const char* VERSION = "@qr_VERSION@";

src/QR.cpp

@@ -14,6 +14,7 @@ namespace massivedynamic {
 
 QR::QR(const std::string& data, std::string outputFile, size_t size, qrcodegen::QrCode::Ecc type) : outputFile(std::move(outputFile)), 
 		qr(qrcodegen::QrCode::encodeText(data.c_str(), type)), size(size) {
+	this->pixels.reserve(this->qr.getSize() * this->qr.getSize());
 	// this is inherently stupid, but "qrcodegen::QrCode" does not give access to the
 	// `segments` vector member and the class itself is marked final.
 	for (int y = 0; y < qr.getSize(); y++) {

src/QR.hpp

@@ -2,15 +2,12 @@
 
 #include <vector>
 #include <string>
+#include <qrcodegen.hpp>
 
-#include "qrcodegen.hpp"
+#include "util.hpp"
 #include "Renderer.hpp"
 
-#define UNUSED_CASE(name) case name: break; 
-
 namespace massivedynamic {
-	typedef uint32_t Color;
-
 	enum class Format {
 		CONSOLE,
 		SVG,

src/util.hpp

@@ -0,0 +1,12 @@
+#pragma once
+
+#include <stdint.h>
+
+#define UNUSED(var) (void) var;
+#define UNUSED_CASE(name) case name: break; 
+
+namespace massivedynamic {
+
+	using Color = uint32_t;
+
+}