From 9f2c8d9e9640c55f1a00785f6bbbe6484a8804c2 Mon Sep 17 00:00:00 2001
From: Project Nayuki <me@nayuki.io>
Date: Fri, 18 Aug 2017 00:17:11 +0000
Subject: [PATCH] Simplified and tweaked various small pieces of C++ code.

---
 cpp/BitBuffer.cpp |  1 -
 cpp/QrCode.cpp    | 18 ++++++++----------
 cpp/QrCode.hpp    |  4 ++--
 cpp/QrSegment.cpp |  1 -
 4 files changed, 10 insertions(+), 14 deletions(-)

diff --git a/cpp/BitBuffer.cpp b/cpp/BitBuffer.cpp
index d8a8357..7f1441a 100644
--- a/cpp/BitBuffer.cpp
+++ b/cpp/BitBuffer.cpp
@@ -21,7 +21,6 @@
  *   Software.
  */
 
-#include <climits>
 #include "BitBuffer.hpp"
 
 
diff --git a/cpp/QrCode.cpp b/cpp/QrCode.cpp
index cd55f03..d9bbc9e 100644
--- a/cpp/QrCode.cpp
+++ b/cpp/QrCode.cpp
@@ -55,8 +55,7 @@ QrCode QrCode::encodeText(const char *text, const Ecc &ecl) {
 
 
 QrCode QrCode::encodeBinary(const vector<uint8_t> &data, const Ecc &ecl) {
-	vector<QrSegment> segs;
-	segs.push_back(QrSegment::makeBytes(data));
+	vector<QrSegment> segs{QrSegment::makeBytes(data)};
 	return encodeSegments(segs, ecl);
 }
 
@@ -97,7 +96,7 @@ QrCode QrCode::encodeSegments(const vector<QrSegment> &segs, const Ecc &ecl,
 	}
 	
 	// Add terminator and pad up to a byte if applicable
-	bb.appendBits(0, std::min(static_cast<size_t>(4), dataCapacityBits - bb.size()));
+	bb.appendBits(0, std::min<size_t>(4, dataCapacityBits - bb.size()));
 	bb.appendBits(0, (8 - bb.size() % 8) % 8);
 	
 	// Pad with alternate bytes until data capacity is reached
@@ -323,14 +322,13 @@ vector<uint8_t> QrCode::appendErrorCorrection(const vector<uint8_t> &data) const
 	vector<vector<uint8_t> > blocks;
 	const ReedSolomonGenerator rs(blockEccLen);
 	for (int i = 0, k = 0; i < numBlocks; i++) {
-		vector<uint8_t> dat;
-		dat.insert(dat.begin(), data.begin() + k, data.begin() + (k + shortBlockLen - blockEccLen + (i < numShortBlocks ? 0 : 1)));
+		vector<uint8_t> dat(data.cbegin() + k, data.cbegin() + (k + shortBlockLen - blockEccLen + (i < numShortBlocks ? 0 : 1)));
 		k += dat.size();
 		const vector<uint8_t> ecc(rs.getRemainder(dat));
 		if (i < numShortBlocks)
 			dat.push_back(0);
-		dat.insert(dat.end(), ecc.begin(), ecc.end());
-		blocks.push_back(dat);
+		dat.insert(dat.end(), ecc.cbegin(), ecc.cend());
+		blocks.push_back(std::move(dat));
 	}
 	
 	// Interleave (not concatenate) the bytes from every block into a single sequence
@@ -487,9 +485,9 @@ long QrCode::getPenaltyScore() const {
 	
 	// Balance of black and white modules
 	int black = 0;
-	for (int y = 0; y < size; y++) {
-		for (int x = 0; x < size; x++) {
-			if (module(x, y))
+	for (const vector<bool> &row : modules) {
+		for (bool color : row) {
+			if (color)
 				black++;
 		}
 	}
diff --git a/cpp/QrCode.hpp b/cpp/QrCode.hpp
index 93fe88c..a9a21df 100644
--- a/cpp/QrCode.hpp
+++ b/cpp/QrCode.hpp
@@ -110,8 +110,8 @@ class QrCode final {
 	private: int mask;
 	
 	// Private grids of modules/pixels (conceptually immutable)
-	private: std::vector<std::vector<bool>> modules;     // The modules of this QR Code symbol (false = white, true = black)
-	private: std::vector<std::vector<bool>> isFunction;  // Indicates function modules that are not subjected to masking
+	private: std::vector<std::vector<bool> > modules;     // The modules of this QR Code symbol (false = white, true = black)
+	private: std::vector<std::vector<bool> > isFunction;  // Indicates function modules that are not subjected to masking
 	
 	
 	
diff --git a/cpp/QrSegment.cpp b/cpp/QrSegment.cpp
index 7e1960b..e65c276 100644
--- a/cpp/QrSegment.cpp
+++ b/cpp/QrSegment.cpp
@@ -22,7 +22,6 @@
  */
 
 #include <climits>
-#include <cstddef>
 #include <cstring>
 #include "QrSegment.hpp"