IWYU pass

src/BC1/BC1Decoder.cpp

@@ -19,14 +19,12 @@
 
 #include "BC1Decoder.h"
 
-#include <assert.h>
-#include <stdint.h>
-
 #include <array>
+#include <cassert>
+#include <cstdint>
 
 #include "../BlockView.h"
 #include "../Color.h"
-#include "../Interpolator.h"
 #include "../ndebug.h"
 #include "BC1Block.h"
 

src/BC3/BC3Decoder.cpp

@@ -23,6 +23,7 @@
 #include "../BC4/BC4Decoder.h"
 #include "../BlockView.h"
 #include "../ndebug.h"
+#include "BC3Block.h"
 
 namespace rgbcx {
 

src/BC4/BC4Decoder.cpp

@@ -19,12 +19,10 @@
 
 #include "BC4Decoder.h"
 
-#include <assert.h>  // for assert
+#include <array>    // for array
-
+#include <cassert>  // for assert
-#include <array>  // for array
 
 #include "../BlockView.h"  // for ColorBlock
-#include "../Color.h"      // for Color
 #include "../ndebug.h"     // for ndebug
 #include "BC4Block.h"
 

src/BC4/BC4Decoder.h

@@ -19,7 +19,8 @@
 
 #pragma once
 
-#include <stddef.h>
+#include <cassert>
+#include <cstdint>
 
 #include "../BlockDecoder.h"
 #include "../BlockView.h"

src/BC4/BC4Encoder.cpp

@@ -19,7 +19,12 @@
 
 #include "BC4Encoder.h"
 
-#include <algorithm>
+#include <algorithm>  // for minmax_element
+#include <array>      // for array
+#include <cstdint>    // for uint8_t
+#include <utility>    // for pair
+
+#include "BC4Block.h"  // for BC4Block
 
 namespace rgbcx {
 void BC4Encoder::EncodeBlock(Byte4x4 pixels, BC4Block *const dest) const noexcept(ndebug) {
@@ -38,7 +43,7 @@ void BC4Encoder::EncodeBlock(Byte4x4 pixels, BC4Block *const dest) const noexcep
     }
 
     std::array<uint8_t, 16> selectors = {};
-    const static std::array<uint8_t, 8> Levels = {1U, 7U, 6U, 5U, 4U, 3U, 2U, 0U}; // selector value options in linear order
+    const static std::array<uint8_t, 8> Levels = {1U, 7U, 6U, 5U, 4U, 3U, 2U, 0U};  // selector value options in linear order
 
     // BC4 floors in its divisions, which we compensate for with the 4 bias.
     // This function is optimal for all possible inputs (i.e. it outputs the same results as checking all 8 values and choosing the closest one).
@@ -52,7 +57,7 @@ void BC4Encoder::EncodeBlock(Byte4x4 pixels, BC4Block *const dest) const noexcep
 
     // iterate over all values and calculate selectors
     for (unsigned i = 0; i < 16; i++) {
-        int value = flattened[i] * 14; // multiply by demonimator
+        int value = flattened[i] * 14;  // multiply by demonimator
 
         // level = number of thresholds this value is greater than
         unsigned level = 0;

src/BC4/BC4Encoder.h

@@ -19,8 +19,14 @@
 
 #pragma once
 
+#include <cassert>
+#include <cstdint>
+
 #include "../BlockEncoder.h"
+#include "../BlockView.h"
+#include "../ndebug.h"
 #include "BC4Block.h"
+
 namespace rgbcx {
 
 class BC4Encoder : public BlockEncoder<BC4Block, 4, 4> {

src/BC5/BC5Decoder.cpp

@@ -19,9 +19,9 @@
 
 #include "BC5Decoder.h"
 
-#include "../BC4/BC4Decoder.h"
 #include "../BlockView.h"
 #include "../ndebug.h"
+#include "BC5Block.h"
 
 namespace rgbcx {
 

src/BC5/BC5Decoder.h

@@ -19,8 +19,9 @@
 
 #pragma once
 
-#include <stddef.h>
+#include <cassert>
-
+#include <cstddef>
+#include <cstdint>
 #include <memory>
 
 #include "../BC4/BC4Decoder.h"