[iwyu] gprpp (#29534)

* [iwyu] gprpp

* Automated change: Fix sanity tests

Co-authored-by: ctiller <ctiller@users.noreply.github.com>

src/core/lib/gprpp/bitset.h

@@ -17,6 +17,10 @@
 
 #include <grpc/support/port_platform.h>
 
+#include <stddef.h>
+#include <stdint.h>
+
+#include <type_traits>
 #include <utility>
 
 #include "src/core/lib/gpr/useful.h"
@@ -33,6 +37,7 @@ namespace grpc_core {
 // exactly that number of bits. Undefined if that bit count is not available.
 template <size_t kBits>
 struct UintSelector;
+
 template <>
 struct UintSelector<8> {
   typedef uint8_t Type;

src/core/lib/gprpp/capture.h

@@ -18,6 +18,7 @@
 #include <grpc/support/port_platform.h>
 
 #include <tuple>
+#include <type_traits>
 #include <utility>
 
 #include "absl/utility/utility.h"

src/core/lib/gprpp/chunked_vector.h

@@ -18,6 +18,10 @@
 #include <grpc/support/port_platform.h>
 
 #include <cstddef>
+#include <iterator>
+#include <utility>
+
+#include <grpc/support/log.h>
 
 #include "src/core/lib/gprpp/manual_constructor.h"
 #include "src/core/lib/resource_quota/arena.h"

src/core/lib/gprpp/construct_destruct.h

@@ -17,6 +17,7 @@
 
 #include <grpc/support/port_platform.h>
 
+#include <new>
 #include <utility>
 
 namespace grpc_core {

src/core/lib/gprpp/dual_ref_counted.h

@@ -22,6 +22,7 @@
 #include <atomic>
 #include <cassert>
 #include <cinttypes>
+#include <cstdint>
 
 #include <grpc/support/atm.h>
 #include <grpc/support/log.h>

src/core/lib/gprpp/fork.cc

@@ -20,15 +20,13 @@
 
 #include "src/core/lib/gprpp/fork.h"
 
-#include <string.h>
-
-#include <grpc/support/alloc.h>
+#include <grpc/impl/codegen/gpr_types.h>
+#include <grpc/support/atm.h>
 #include <grpc/support/sync.h>
 #include <grpc/support/time.h>
 
-#include "src/core/lib/gpr/useful.h"
-#include "src/core/lib/gprpp/global_config.h"
-#include "src/core/lib/gprpp/memory.h"
+#include "src/core/lib/gprpp/global_config_env.h"
+#include "src/core/lib/gprpp/global_config_generic.h"
 
 /*
  * NOTE: FORKING IS NOT GENERALLY SUPPORTED, THIS IS ONLY INTENDED TO WORK

src/core/lib/gprpp/global_config_env.cc

@@ -21,13 +21,14 @@
 #include "src/core/lib/gprpp/global_config_env.h"
 
 #include <ctype.h>
-#include <string.h>
+#include <stdlib.h>
 
+#include <memory>
 #include <string>
+#include <type_traits>
 
 #include "absl/strings/str_format.h"
 
-#include <grpc/support/alloc.h>
 #include <grpc/support/log.h>
 #include <grpc/support/string_util.h>
 

src/core/lib/gprpp/global_config_env.h

@@ -21,6 +21,8 @@
 
 #include <grpc/support/port_platform.h>
 
+#include <stdint.h>
+
 #include "src/core/lib/gprpp/global_config_generic.h"
 #include "src/core/lib/gprpp/memory.h"
 

src/core/lib/gprpp/host_port.cc

@@ -20,6 +20,8 @@
 
 #include "src/core/lib/gprpp/host_port.h"
 
+#include <stddef.h>
+
 #include "absl/strings/str_format.h"
 #include "absl/strings/string_view.h"
 

src/core/lib/gprpp/match.h

@@ -17,6 +17,8 @@
 
 #include <grpc/support/port_platform.h>
 
+#include <type_traits>
+
 #include "absl/types/variant.h"
 
 #include "src/core/lib/gprpp/overload.h"

src/core/lib/gprpp/memory.h

@@ -23,6 +23,7 @@
 
 #include <limits>
 #include <memory>
+#include <type_traits>
 #include <utility>
 
 #include "absl/memory/memory.h"

src/core/lib/gprpp/orphanable.h

@@ -23,6 +23,7 @@
 
 #include <cinttypes>
 #include <memory>
+#include <utility>
 
 #include <grpc/support/log.h>
 #include <grpc/support/sync.h>

src/core/lib/gprpp/ref_counted.h

@@ -234,6 +234,7 @@ enum UnrefBehavior {
 namespace internal {
 template <typename T, UnrefBehavior UnrefBehaviorArg>
 class Delete;
+
 template <typename T>
 class Delete<T, kUnrefDelete> {
  public:

src/core/lib/gprpp/ref_counted_ptr.h

@@ -21,6 +21,7 @@
 
 #include <grpc/support/port_platform.h>
 
+#include <iosfwd>
 #include <type_traits>
 #include <utility>
 

src/core/lib/gprpp/stat_posix.cc

@@ -16,12 +16,15 @@
 
 #include <grpc/support/port_platform.h>
 
+#include <string.h>
+
+#include "absl/status/status.h"
+#include "absl/strings/string_view.h"
+
 #ifdef GPR_POSIX_STAT
 
 #include <errno.h>
 #include <sys/stat.h>
-#include <sys/types.h>
-#include <unistd.h>
 
 #include <grpc/support/log.h>
 

src/core/lib/gprpp/status_helper.cc

@@ -20,22 +20,26 @@
 
 #include "src/core/lib/gprpp/status_helper.h"
 
-#include <type_traits>
+#include <string.h>
+
+#include <algorithm>
+#include <new>
+#include <utility>
 
 #include "absl/strings/cord.h"
 #include "absl/strings/escaping.h"
 #include "absl/strings/match.h"
-#include "absl/strings/str_format.h"
+#include "absl/strings/numbers.h"
+#include "absl/strings/str_cat.h"
 #include "absl/strings/str_join.h"
 #include "absl/time/clock.h"
 #include "google/protobuf/any.upb.h"
 #include "google/rpc/status.upb.h"
+#include "upb/upb.h"
 #include "upb/upb.hpp"
 
 #include <grpc/support/log.h>
 
-#include "src/core/lib/gprpp/time_util.h"
-
 namespace grpc_core {
 
 namespace {

src/core/lib/gprpp/status_helper.h

@@ -21,9 +21,13 @@
 
 #include <grpc/support/port_platform.h>
 
+#include <stdint.h>
+
+#include <string>
 #include <vector>
 
 #include "absl/status/status.h"
+#include "absl/strings/string_view.h"
 #include "absl/time/time.h"
 #include "absl/types/optional.h"
 

src/core/lib/gprpp/sync.h

@@ -21,6 +21,7 @@
 
 #include <grpc/support/port_platform.h>
 
+#include "absl/base/thread_annotations.h"
 #include "absl/synchronization/mutex.h"
 
 #include <grpc/support/log.h>

src/core/lib/gprpp/table.h

@@ -17,8 +17,14 @@
 
 #include <grpc/support/port_platform.h>
 
+#include <stddef.h>
+
+#include <initializer_list>
+#include <new>
+#include <type_traits>
 #include <utility>
 
+#include "absl/meta/type_traits.h"
 #include "absl/utility/utility.h"
 
 #include "src/core/lib/gprpp/bitset.h"
@@ -31,6 +37,7 @@ namespace table_detail {
 // A tuple-like type that contains manually constructed elements.
 template <typename... Ts>
 struct Elements;
+
 template <typename T, typename... Ts>
 struct Elements<T, Ts...> : Elements<Ts...> {
   union U {
@@ -71,6 +78,7 @@ struct GetElem<I, T, Ts...> {
 // Sets a member constant N to the number of times Needle is in Haystack.
 template <typename Needle, typename... Haystack>
 struct CountIncludedStruct;
+
 template <typename Needle, typename Straw, typename... RestOfHaystack>
 struct CountIncludedStruct<Needle, Straw, RestOfHaystack...> {
   static constexpr size_t N =
@@ -92,6 +100,7 @@ constexpr size_t CountIncluded() {
 // Ignored should be void always, and is used for enable_if_t.
 template <typename Ignored, typename Needle, typename... Haystack>
 struct IndexOfStruct;
+
 template <typename Needle, typename Straw, typename... RestOfHaystack>
 struct IndexOfStruct<absl::enable_if_t<std::is_same<Needle, Straw>::value>,
                      Needle, Straw, RestOfHaystack...> {
@@ -123,6 +132,7 @@ IndexOf() {
 // Implemented as a simple type recursion.
 template <size_t I, typename... Ts>
 struct TypeIndexStruct;
+
 template <typename T, typename... Ts>
 struct TypeIndexStruct<0, T, Ts...> {
   using Type = T;

src/core/lib/gprpp/thd.h

@@ -23,6 +23,8 @@
 
 #include <grpc/support/port_platform.h>
 
+#include <stddef.h>
+
 #include <grpc/support/log.h>
 #include <grpc/support/sync.h>
 #include <grpc/support/thd_id.h>

src/core/lib/gprpp/thd_posix.cc

@@ -20,6 +20,9 @@
 
 #include <grpc/support/port_platform.h>
 
+#include <grpc/impl/codegen/gpr_types.h>
+#include <grpc/support/time.h>
+
 #ifdef GPR_POSIX_SYNC
 
 #include <pthread.h>
@@ -27,19 +30,18 @@
 #include <string.h>
 #include <unistd.h>
 
-#include <grpc/support/alloc.h>
 #include <grpc/support/log.h>
 #include <grpc/support/sync.h>
 #include <grpc/support/thd_id.h>
 
 #include "src/core/lib/gpr/useful.h"
 #include "src/core/lib/gprpp/fork.h"
-#include "src/core/lib/gprpp/memory.h"
 #include "src/core/lib/gprpp/thd.h"
 
 namespace grpc_core {
 namespace {
 class ThreadInternalsPosix;
+
 struct thd_arg {
   ThreadInternalsPosix* thread;
   void (*body)(void* arg); /* body of a thread */

src/core/lib/gprpp/time.cc

@@ -20,6 +20,7 @@
 #include <cstdint>
 #include <limits>
 #include <string>
+#include <utility>
 
 #include "absl/strings/str_format.h"
 

src/core/lib/gprpp/time.h

@@ -24,6 +24,7 @@
 #include <ostream>
 #include <string>
 
+#include <grpc/impl/codegen/gpr_types.h>
 #include <grpc/support/time.h>
 
 #include "src/core/lib/gpr/time_precise.h"

src/core/lib/gprpp/time_util.cc

@@ -18,7 +18,11 @@
 
 #include "src/core/lib/gprpp/time_util.h"
 
+#include <stdint.h>
+#include <time.h>
+
 #include <grpc/support/log.h>
+#include <grpc/support/time.h>
 
 namespace grpc_core {
 

src/core/lib/gprpp/time_util.h

@@ -21,6 +21,7 @@
 
 #include "absl/time/time.h"
 
+#include <grpc/impl/codegen/gpr_types.h>
 #include <grpc/support/time.h>
 
 namespace grpc_core {

tools/dockerfile/grpc_iwyu/iwyu.sh

@@ -32,6 +32,7 @@ export ENABLED_MODULES='
   src/core/lib/avl
   src/core/lib/channel
   src/core/lib/config
+  src/core/lib/gprpp
   src/core/lib/json
   src/core/lib/slice
   src/core/lib/resource_quota