From aebb5329068e7c835b08446f028a1c764f57393c Mon Sep 17 00:00:00 2001 From: Kefu Chai Date: Fri, 8 Nov 2024 11:13:09 +0800 Subject: [PATCH] bytes, utils: include fmt/iostream.h and iostream when appropriate in seastar e96932b05f394b27cd0101e24f0584736795b50f, we stopped including unused `fmt/ostream.h`. this helped to reduce the header dependency. but this also broke the build of scylladb, as we rely on the `fmt/ostream.h` indirectly included by seastar's header project. in this change, we include `fmt/iostream.h` and `iostream` explictly when we are using the declarations in them. this enables us to - bump up the seastar submodule - potentially reduce the header dependency as we will be able to include seastar/core/format.hh instead of a more bloated seastar/core/print.hh after bumping up seastar submodule Signed-off-by: Kefu Chai Closes scylladb/scylladb#21494 --- bytes.cc | 1 + utils/estimated_histogram.hh | 1 + utils/rjson.hh | 1 + utils/uuid.cc | 1 + 4 files changed, 4 insertions(+) diff --git a/bytes.cc b/bytes.cc index 1f2197a2f9..5dccd34ffd 100644 --- a/bytes.cc +++ b/bytes.cc @@ -7,6 +7,7 @@ */ #include "bytes.hh" +#include #include static inline int8_t hex_to_int(unsigned char c) { diff --git a/utils/estimated_histogram.hh b/utils/estimated_histogram.hh index e6eefb0049..de1919b006 100644 --- a/utils/estimated_histogram.hh +++ b/utils/estimated_histogram.hh @@ -15,6 +15,7 @@ #include #include #include +#include #include #include #include "seastarx.hh" diff --git a/utils/rjson.hh b/utils/rjson.hh index 2b5a6f0a6e..b78eea6b4e 100644 --- a/utils/rjson.hh +++ b/utils/rjson.hh @@ -26,6 +26,7 @@ * or calling Size() on a non-array value. */ +#include #include #include #include diff --git a/utils/uuid.cc b/utils/uuid.cc index ce38835c15..55f46033f5 100644 --- a/utils/uuid.cc +++ b/utils/uuid.cc @@ -13,6 +13,7 @@ #include #include #include +#include #include #include "marshal_exception.hh"