From c22323afda89c0800d8ba998d502e890bc97cab1 Mon Sep 17 00:00:00 2001 From: Lee Rhodes Date: Mon, 28 Sep 2026 16:51:28 -0700 Subject: [PATCH] Add missing standard includes to hll and cpc internal headers coupon_iterator.hpp uses uint32_t and std::input_iterator_tag, and compression_data.hpp uses uint8_t and uint16_t, but neither includes the headers that declare them. Both compile today only because of include order. This is the same latent problem as reverse_purge_hash_map.hpp in #499, which surfaced under GCC 15's reduced transitive includes. Co-Authored-By: Claude Opus 5.5 --- cpc/include/compression_data.hpp | 2 ++ hll/include/coupon_iterator.hpp | 3 +++ 2 files changed, 5 insertions(+) diff --git a/cpc/include/compression_data.hpp b/cpc/include/compression_data.hpp index 0889b479..8444fa72 100644 --- a/cpc/include/compression_data.hpp +++ b/cpc/include/compression_data.hpp @@ -22,6 +22,8 @@ #ifndef CPC_COMPRESSION_DATA_HPP_ #define CPC_COMPRESSION_DATA_HPP_ +#include + namespace datasketches { /* diff --git a/hll/include/coupon_iterator.hpp b/hll/include/coupon_iterator.hpp index cd62c827..fff56cd2 100644 --- a/hll/include/coupon_iterator.hpp +++ b/hll/include/coupon_iterator.hpp @@ -20,6 +20,9 @@ #ifndef _INTARRAYPAIRITERATOR_HPP_ #define _INTARRAYPAIRITERATOR_HPP_ +#include +#include + namespace datasketches { template