From 2d5bf532403058885f56fc62081b5099be5d09a3 Mon Sep 17 00:00:00 2001
From: Max Kellermann <max@musicpd.org>
Date: Tue, 14 Jan 2020 22:39:12 +0100
Subject: [PATCH] pcm/Dsd2Pcm: use `sizet_t` instead of `int`

---
 src/pcm/Dsd2Pcm.cxx | 7 ++++---
 src/pcm/Dsd2Pcm.hxx | 2 +-
 2 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/src/pcm/Dsd2Pcm.cxx b/src/pcm/Dsd2Pcm.cxx
index aafb524a2..db1da9159 100644
--- a/src/pcm/Dsd2Pcm.cxx
+++ b/src/pcm/Dsd2Pcm.cxx
@@ -41,7 +41,7 @@ or implied, of Sebastian Gesemann.
 static constexpr size_t HTAPS = 48;
 
 /** number of "8 MACs" lookup tables */
-static constexpr int CTABLES = (HTAPS + 7) / 8;
+static constexpr size_t CTABLES = (HTAPS + 7) / 8;
 
 static_assert(Dsd2Pcm::FIFOSIZE * 8 >= HTAPS * 2, "FIFOSIZE too small");
 
@@ -117,7 +117,7 @@ static constexpr double htaps[HTAPS] = {
 };
 
 static constexpr float
-CalculateCtableValue(int t, int k, int e) noexcept
+CalculateCtableValue(size_t t, int k, int e) noexcept
 {
 	double acc = 0;
 	for (int m = 0; m < k; ++m) {
@@ -130,7 +130,8 @@ CalculateCtableValue(int t, int k, int e) noexcept
 /* this needs to be a struct because GCC 6 doesn't have constexpr
    lambdas (C++17) */
 struct GenerateCtableValue {
-	int t, k;
+	size_t t;
+	int k;
 
 	constexpr auto operator()(int e) const noexcept {
 		return CalculateCtableValue(t, k, e);
diff --git a/src/pcm/Dsd2Pcm.hxx b/src/pcm/Dsd2Pcm.hxx
index 027a004a3..55d027e38 100644
--- a/src/pcm/Dsd2Pcm.hxx
+++ b/src/pcm/Dsd2Pcm.hxx
@@ -40,7 +40,7 @@ or implied, of Sebastian Gesemann.
 class Dsd2Pcm {
 public:
 	/* must be a power of two */
-	static constexpr int FIFOSIZE = 16;
+	static constexpr size_t FIFOSIZE = 16;
 
 private:
 	/** bit mask for FIFO offsets */