grpc 1.63.2 → 1.64.0.pre1

data/src/core/lib/gprpp/host_port.cc

@@ -16,16 +16,16 @@
 //
 //
 
-#include <grpc/support/port_platform.h>
-
 #include "src/core/lib/gprpp/host_port.h"
 
 #include <stddef.h>
 
+#include "absl/log/check.h"
 #include "absl/strings/str_format.h"
 #include "absl/strings/string_view.h"
 
 #include <grpc/support/log.h>
+#include <grpc/support/port_platform.h>
 
 namespace grpc_core {
 
@@ -93,8 +93,10 @@ bool SplitHostPort(absl::string_view name, absl::string_view* host,
 
 bool SplitHostPort(absl::string_view name, std::string* host,
                    std::string* port) {
-  GPR_DEBUG_ASSERT(host != nullptr && host->empty());
-  GPR_DEBUG_ASSERT(port != nullptr && port->empty());
+  DCHECK(host != nullptr);
+  DCHECK(host->empty());
+  DCHECK(port != nullptr);
+  DCHECK(port->empty());
   absl::string_view host_view;
   absl::string_view port_view;
   bool has_port;

data/src/core/lib/gprpp/host_port.h

@@ -19,12 +19,12 @@
 #ifndef GRPC_SRC_CORE_LIB_GPRPP_HOST_PORT_H
 #define GRPC_SRC_CORE_LIB_GPRPP_HOST_PORT_H
 
-#include <grpc/support/port_platform.h>
-
 #include <string>
 
 #include "absl/strings/string_view.h"
 
+#include <grpc/support/port_platform.h>
+
 namespace grpc_core {
 
 // Given a host and port, creates a newly-allocated string of the form

data/src/core/lib/gprpp/linux/env.cc

@@ -21,12 +21,12 @@
 #define _GNU_SOURCE
 #endif
 
-#include <grpc/support/port_platform.h>
-
 #include <string>
 
 #include "absl/types/optional.h"
 
+#include <grpc/support/port_platform.h>
+
 #ifdef GPR_LINUX_ENV
 
 #include <features.h>

data/src/core/lib/gprpp/load_file.cc

@@ -12,8 +12,6 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-#include <grpc/support/port_platform.h>
-
 #include "src/core/lib/gprpp/load_file.h"
 
 #include <errno.h>
@@ -27,6 +25,7 @@
 #include <grpc/slice.h>
 #include <grpc/support/alloc.h>
 #include <grpc/support/log.h>
+#include <grpc/support/port_platform.h>
 
 namespace grpc_core {
 

data/src/core/lib/gprpp/load_file.h

@@ -15,12 +15,12 @@
 #ifndef GRPC_SRC_CORE_LIB_GPRPP_LOAD_FILE_H
 #define GRPC_SRC_CORE_LIB_GPRPP_LOAD_FILE_H
 
-#include <grpc/support/port_platform.h>
-
 #include <string>
 
 #include "absl/status/statusor.h"
 
+#include <grpc/support/port_platform.h>
+
 #include "src/core/lib/slice/slice.h"
 
 namespace grpc_core {

data/src/core/lib/gprpp/manual_constructor.h

@@ -21,12 +21,12 @@
 
 // manually construct a region of memory with some type
 
-#include <grpc/support/port_platform.h>
-
 #include <stddef.h>
 
 #include <utility>
 
+#include <grpc/support/port_platform.h>
+
 #include "src/core/lib/gprpp/construct_destruct.h"
 
 namespace grpc_core {

data/src/core/lib/gprpp/match.h

@@ -15,12 +15,12 @@
 #ifndef GRPC_SRC_CORE_LIB_GPRPP_MATCH_H
 #define GRPC_SRC_CORE_LIB_GPRPP_MATCH_H
 
-#include <grpc/support/port_platform.h>
-
 #include <utility>
 
 #include "absl/types/variant.h"
 
+#include <grpc/support/port_platform.h>
+
 #include "src/core/lib/gprpp/overload.h"
 
 namespace grpc_core {

data/src/core/lib/gprpp/memory.h

@@ -19,12 +19,11 @@
 #ifndef GRPC_SRC_CORE_LIB_GPRPP_MEMORY_H
 #define GRPC_SRC_CORE_LIB_GPRPP_MEMORY_H
 
-#include <grpc/support/port_platform.h>
-
 #include <memory>
 #include <type_traits>
 
 #include <grpc/support/alloc.h>
+#include <grpc/support/port_platform.h>
 
 namespace grpc_core {
 

data/src/core/lib/gprpp/mpscq.cc

@@ -16,10 +16,10 @@
 //
 //
 
-#include <grpc/support/port_platform.h>
-
 #include "src/core/lib/gprpp/mpscq.h"
 
+#include <grpc/support/port_platform.h>
+
 namespace grpc_core {
 
 //

data/src/core/lib/gprpp/mpscq.h

@@ -19,11 +19,12 @@
 #ifndef GRPC_SRC_CORE_LIB_GPRPP_MPSCQ_H
 #define GRPC_SRC_CORE_LIB_GPRPP_MPSCQ_H
 
-#include <grpc/support/port_platform.h>
-
 #include <atomic>
 
+#include "absl/log/check.h"
+
 #include <grpc/support/log.h>
+#include <grpc/support/port_platform.h>
 
 #include "src/core/lib/gprpp/sync.h"
 
@@ -41,8 +42,8 @@ class MultiProducerSingleConsumerQueue {
 
   MultiProducerSingleConsumerQueue() : head_{&stub_}, tail_(&stub_) {}
   ~MultiProducerSingleConsumerQueue() {
-    GPR_ASSERT(head_.load(std::memory_order_relaxed) == &stub_);
-    GPR_ASSERT(tail_ == &stub_);
+    CHECK(head_.load(std::memory_order_relaxed) == &stub_);
+    CHECK(tail_ == &stub_);
   }
 
   // Push a node

data/src/core/lib/gprpp/no_destruct.h

@@ -15,11 +15,11 @@
 #ifndef GRPC_SRC_CORE_LIB_GPRPP_NO_DESTRUCT_H
 #define GRPC_SRC_CORE_LIB_GPRPP_NO_DESTRUCT_H
 
-#include <grpc/support/port_platform.h>
-
 #include <type_traits>
 #include <utility>
 
+#include <grpc/support/port_platform.h>
+
 #include "src/core/lib/gprpp/construct_destruct.h"
 
 namespace grpc_core {

data/src/core/lib/gprpp/notification.h

@@ -15,11 +15,11 @@
 #ifndef GRPC_SRC_CORE_LIB_GPRPP_NOTIFICATION_H
 #define GRPC_SRC_CORE_LIB_GPRPP_NOTIFICATION_H
 
-#include <grpc/support/port_platform.h>
-
 #include "absl/time/clock.h"
 #include "absl/time/time.h"
 
+#include <grpc/support/port_platform.h>
+
 #include "src/core/lib/gprpp/sync.h"
 
 namespace grpc_core {

data/src/core/lib/gprpp/orphanable.h

@@ -19,12 +19,12 @@
 #ifndef GRPC_SRC_CORE_LIB_GPRPP_ORPHANABLE_H
 #define GRPC_SRC_CORE_LIB_GPRPP_ORPHANABLE_H
 
-#include <grpc/support/port_platform.h>
-
 #include <cinttypes>
 #include <memory>
 #include <utility>
 
+#include <grpc/support/port_platform.h>
+
 #include "src/core/lib/gprpp/debug_location.h"
 #include "src/core/lib/gprpp/down_cast.h"
 #include "src/core/lib/gprpp/ref_counted.h"

data/src/core/lib/gprpp/overload.h

@@ -15,10 +15,10 @@
 #ifndef GRPC_SRC_CORE_LIB_GPRPP_OVERLOAD_H
 #define GRPC_SRC_CORE_LIB_GPRPP_OVERLOAD_H
 
-#include <grpc/support/port_platform.h>
-
 #include <utility>
 
+#include <grpc/support/port_platform.h>
+
 namespace grpc_core {
 
 template <typename... Cases>

data/src/core/lib/gprpp/per_cpu.cc

@@ -12,11 +12,10 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-#include <grpc/support/port_platform.h>
-
 #include "src/core/lib/gprpp/per_cpu.h"
 
 #include <grpc/support/cpu.h>
+#include <grpc/support/port_platform.h>
 
 #include "src/core/lib/gpr/useful.h"
 

data/src/core/lib/gprpp/per_cpu.h

@@ -15,8 +15,6 @@
 #ifndef GRPC_SRC_CORE_LIB_GPRPP_PER_CPU_H
 #define GRPC_SRC_CORE_LIB_GPRPP_PER_CPU_H
 
-#include <grpc/support/port_platform.h>
-
 #include <stdint.h>
 
 #include <algorithm>
@@ -25,6 +23,7 @@
 #include <memory>
 
 #include <grpc/support/cpu.h>
+#include <grpc/support/port_platform.h>
 
 // Sharded collections of objects
 // This used to be per-cpu, now it's much less so - but still a way to limit

data/src/core/lib/gprpp/posix/directory_reader.cc

@@ -16,14 +16,14 @@
 //
 //
 
-#include <grpc/support/port_platform.h>
-
 #include <memory>
 
 #include "absl/functional/function_ref.h"
 #include "absl/status/status.h"
 #include "absl/strings/string_view.h"
 
+#include <grpc/support/port_platform.h>
+
 #if defined(GPR_LINUX) || defined(GPR_ANDROID) || defined(GPR_FREEBSD) || \
     defined(GPR_APPLE)
 

data/src/core/lib/gprpp/posix/stat.cc

@@ -14,10 +14,10 @@
 // limitations under the License.
 //
 
-#include <grpc/support/port_platform.h>
-
 #include <string>
 
+#include <grpc/support/port_platform.h>
+
 // IWYU pragma: no_include <bits/struct_stat.h>
 
 #include "absl/status/status.h"
@@ -27,6 +27,8 @@
 #include <errno.h>
 #include <sys/stat.h>
 
+#include "absl/log/check.h"
+
 #include <grpc/support/log.h>
 
 #include "src/core/lib/gprpp/stat.h"
@@ -35,8 +37,8 @@
 namespace grpc_core {
 
 absl::Status GetFileModificationTime(const char* filename, time_t* timestamp) {
-  GPR_ASSERT(filename != nullptr);
-  GPR_ASSERT(timestamp != nullptr);
+  CHECK_NE(filename, nullptr);
+  CHECK_NE(timestamp, nullptr);
   struct stat buf;
   if (stat(filename, &buf) != 0) {
     std::string error_msg = StrError(errno);

data/src/core/lib/gprpp/posix/thd.cc

@@ -18,13 +18,13 @@
 
 // Posix implementation for gpr threads.
 
-#include <grpc/support/port_platform.h>
-
 #include <inttypes.h>
 
 #include <csignal>
 #include <string>
 
+#include <grpc/support/port_platform.h>
+
 #ifdef GPR_POSIX_SYNC
 
 #include <pthread.h>
@@ -32,6 +32,8 @@
 #include <string.h>
 #include <unistd.h>
 
+#include "absl/log/check.h"
+
 #include <grpc/support/log.h>
 #include <grpc/support/sync.h>
 #include <grpc/support/thd_id.h>
@@ -88,7 +90,7 @@ class ThreadInternalsPosix : public internal::ThreadInternalsInterface {
     // don't use gpr_malloc as we may cause an infinite recursion with
     // the profiling code
     thd_arg* info = static_cast<thd_arg*>(malloc(sizeof(*info)));
-    GPR_ASSERT(info != nullptr);
+    CHECK_NE(info, nullptr);
     info->thread = this;
     info->body = thd_body;
     info->arg = arg;
@@ -99,18 +101,16 @@ class ThreadInternalsPosix : public internal::ThreadInternalsInterface {
       Fork::IncThreadCount();
     }
 
-    GPR_ASSERT(pthread_attr_init(&attr) == 0);
+    CHECK_EQ(pthread_attr_init(&attr), 0);
     if (options.joinable()) {
-      GPR_ASSERT(pthread_attr_setdetachstate(&attr, PTHREAD_CREATE_JOINABLE) ==
-                 0);
+      CHECK(pthread_attr_setdetachstate(&attr, PTHREAD_CREATE_JOINABLE) == 0);
     } else {
-      GPR_ASSERT(pthread_attr_setdetachstate(&attr, PTHREAD_CREATE_DETACHED) ==
-                 0);
+      CHECK(pthread_attr_setdetachstate(&attr, PTHREAD_CREATE_DETACHED) == 0);
     }
 
     if (options.stack_size() != 0) {
       size_t stack_size = MinValidStackSize(options.stack_size());
-      GPR_ASSERT(pthread_attr_setstacksize(&attr, stack_size) == 0);
+      CHECK_EQ(pthread_attr_setstacksize(&attr, stack_size), 0);
     }
 
     int pthread_create_err = pthread_create(
@@ -154,7 +154,7 @@ class ThreadInternalsPosix : public internal::ThreadInternalsInterface {
         info);
     *success = (pthread_create_err == 0);
 
-    GPR_ASSERT(pthread_attr_destroy(&attr) == 0);
+    CHECK_EQ(pthread_attr_destroy(&attr), 0);
 
     if (!(*success)) {
       gpr_log(GPR_ERROR, "pthread_create failed: %s",

data/src/core/lib/gprpp/ref_counted.h

@@ -19,13 +19,14 @@
 #ifndef GRPC_SRC_CORE_LIB_GPRPP_REF_COUNTED_H
 #define GRPC_SRC_CORE_LIB_GPRPP_REF_COUNTED_H
 
-#include <grpc/support/port_platform.h>
-
 #include <atomic>
 #include <cassert>
 #include <cinttypes>
 
+#include "absl/log/check.h"
+
 #include <grpc/support/log.h>
+#include <grpc/support/port_platform.h>
 
 #include "src/core/lib/gprpp/atomic_utils.h"
 #include "src/core/lib/gprpp/debug_location.h"
@@ -165,7 +166,7 @@ class RefCount {
       gpr_log(GPR_INFO, "%s:%p unref %" PRIdPTR " -> %" PRIdPTR, trace, this,
               prior, prior - 1);
     }
-    GPR_DEBUG_ASSERT(prior > 0);
+    DCHECK_GT(prior, 0);
 #endif
     return prior == 1;
   }
@@ -183,7 +184,7 @@ class RefCount {
               trace, this, location.file(), location.line(), prior, prior - 1,
               reason);
     }
-    GPR_DEBUG_ASSERT(prior > 0);
+    DCHECK_GT(prior, 0);
 #else
     // Avoid unused-parameter warnings for debug-only parameters
     (void)location;

data/src/core/lib/gprpp/ref_counted_ptr.h

@@ -19,8 +19,6 @@
 #ifndef GRPC_SRC_CORE_LIB_GPRPP_REF_COUNTED_PTR_H
 #define GRPC_SRC_CORE_LIB_GPRPP_REF_COUNTED_PTR_H
 
-#include <grpc/support/port_platform.h>
-
 #include <stddef.h>
 
 #include <iosfwd>
@@ -29,6 +27,8 @@
 
 #include "absl/hash/hash.h"
 
+#include <grpc/support/port_platform.h>
+
 #include "src/core/lib/gprpp/debug_location.h"
 #include "src/core/lib/gprpp/down_cast.h"
 

data/src/core/lib/gprpp/ref_counted_string.cc

@@ -14,8 +14,6 @@
 // limitations under the License.
 //
 
-#include <grpc/support/port_platform.h>
-
 #include "src/core/lib/gprpp/ref_counted_string.h"
 
 #include <string.h>
@@ -23,6 +21,7 @@
 #include <new>
 
 #include <grpc/support/alloc.h>
+#include <grpc/support/port_platform.h>
 
 namespace grpc_core {
 

data/src/core/lib/gprpp/ref_counted_string.h

@@ -17,14 +17,14 @@
 #ifndef GRPC_SRC_CORE_LIB_GPRPP_REF_COUNTED_STRING_H
 #define GRPC_SRC_CORE_LIB_GPRPP_REF_COUNTED_STRING_H
 
-#include <grpc/support/port_platform.h>
-
 #include <stddef.h>
 
 #include <string>
 
 #include "absl/strings/string_view.h"
 
+#include <grpc/support/port_platform.h>
+
 #include "src/core/lib/gprpp/ref_counted.h"
 #include "src/core/lib/gprpp/ref_counted_ptr.h"
 

data/src/core/lib/gprpp/single_set_ptr.h

@@ -15,12 +15,13 @@
 #ifndef GRPC_SRC_CORE_LIB_GPRPP_SINGLE_SET_PTR_H
 #define GRPC_SRC_CORE_LIB_GPRPP_SINGLE_SET_PTR_H
 
-#include <grpc/support/port_platform.h>
-
 #include <atomic>
 #include <memory>
 
+#include "absl/log/check.h"
+
 #include <grpc/support/log.h>
+#include <grpc/support/port_platform.h>
 
 namespace grpc_core {
 
@@ -68,7 +69,7 @@ class SingleSetPtr {
 
   T* operator->() const {
     T* p = p_.load(std::memory_order_acquire);
-    GPR_DEBUG_ASSERT(p != nullptr);
+    DCHECK_NE(p, nullptr);
     return p;
   }
 

data/src/core/lib/gprpp/sorted_pack.h

@@ -15,10 +15,10 @@
 #ifndef GRPC_SRC_CORE_LIB_GPRPP_SORTED_PACK_H
 #define GRPC_SRC_CORE_LIB_GPRPP_SORTED_PACK_H
 
-#include <grpc/support/port_platform.h>
-
 #include <type_traits>
 
+#include <grpc/support/port_platform.h>
+
 #include "src/core/lib/gprpp/type_list.h"
 
 namespace grpc_core {

data/src/core/lib/gprpp/stat.h

@@ -17,12 +17,12 @@
 #ifndef GRPC_SRC_CORE_LIB_GPRPP_STAT_H
 #define GRPC_SRC_CORE_LIB_GPRPP_STAT_H
 
-#include <grpc/support/port_platform.h>
-
 #include <time.h>
 
 #include "absl/status/status.h"
 
+#include <grpc/support/port_platform.h>
+
 namespace grpc_core {
 
 // Gets the last-modified timestamp of a file or a directory.

data/src/core/lib/gprpp/status_helper.cc

@@ -16,14 +16,13 @@
 //
 //
 
-#include <grpc/support/port_platform.h>
-
 #include "src/core/lib/gprpp/status_helper.h"
 
 #include <string.h>
 
 #include <utility>
 
+#include "absl/log/check.h"
 #include "absl/strings/cord.h"
 #include "absl/strings/escaping.h"
 #include "absl/strings/match.h"
@@ -37,6 +36,7 @@
 #include "upb/mem/arena.hpp"
 
 #include <grpc/support/log.h>
+#include <grpc/support/port_platform.h>
 
 #include "src/core/lib/slice/percent_encoding.h"
 #include "src/core/lib/slice/slice.h"
@@ -154,7 +154,7 @@ std::vector<absl::Status> ParseChildren(absl::Cord children) {
   while (buf.size() - cur >= sizeof(uint32_t)) {
     size_t msg_size = DecodeUInt32FromBytes(buf.data() + cur);
     cur += sizeof(uint32_t);
-    GPR_ASSERT(buf.size() - cur >= msg_size);
+    CHECK(buf.size() - cur >= msg_size);
     google_rpc_Status* msg =
         google_rpc_Status_parse(buf.data() + cur, msg_size, arena.ptr());
     cur += msg_size;

data/src/core/lib/gprpp/status_helper.h

@@ -19,8 +19,6 @@
 #ifndef GRPC_SRC_CORE_LIB_GPRPP_STATUS_HELPER_H
 #define GRPC_SRC_CORE_LIB_GPRPP_STATUS_HELPER_H
 
-#include <grpc/support/port_platform.h>
-
 #include <stdint.h>
 
 #include <string>
@@ -31,6 +29,8 @@
 #include "absl/time/time.h"
 #include "absl/types/optional.h"
 
+#include <grpc/support/port_platform.h>
+
 #include "src/core/lib/gprpp/debug_location.h"
 
 extern "C" {

data/src/core/lib/gprpp/strerror.cc

@@ -12,14 +12,14 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-#include <grpc/support/port_platform.h>
-
 #include "src/core/lib/gprpp/strerror.h"
 
 #include <string.h>
 
 #include "absl/strings/str_format.h"
 
+#include <grpc/support/port_platform.h>
+
 namespace grpc_core {
 
 #ifdef GPR_WINDOWS

data/src/core/lib/gprpp/strerror.h

@@ -15,10 +15,10 @@
 #ifndef GRPC_SRC_CORE_LIB_GPRPP_STRERROR_H
 #define GRPC_SRC_CORE_LIB_GPRPP_STRERROR_H
 
-#include <grpc/support/port_platform.h>
-
 #include <string>
 
+#include <grpc/support/port_platform.h>
+
 namespace grpc_core {
 
 // Returns a string describing the posix error code.

data/src/core/lib/gprpp/sync.h

@@ -19,12 +19,12 @@
 #ifndef GRPC_SRC_CORE_LIB_GPRPP_SYNC_H
 #define GRPC_SRC_CORE_LIB_GPRPP_SYNC_H
 
-#include <grpc/support/port_platform.h>
-
 #include "absl/base/thread_annotations.h"
+#include "absl/log/check.h"
 #include "absl/synchronization/mutex.h"
 
 #include <grpc/support/log.h>
+#include <grpc/support/port_platform.h>
 #include <grpc/support/sync.h>
 
 #ifndef GPR_ABSEIL_SYNC
@@ -113,7 +113,7 @@ class ABSL_SCOPED_LOCKABLE ReleasableMutexLock {
   ReleasableMutexLock& operator=(const ReleasableMutexLock&) = delete;
 
   void Release() ABSL_UNLOCK_FUNCTION() {
-    GPR_DEBUG_ASSERT(!released_);
+    DCHECK(!released_);
     released_ = true;
     mu_->Unlock();
   }
@@ -179,13 +179,13 @@ class ABSL_SCOPED_LOCKABLE LockableAndReleasableMutexLock {
       const LockableAndReleasableMutexLock&) = delete;
 
   void Lock() ABSL_EXCLUSIVE_LOCK_FUNCTION() {
-    GPR_DEBUG_ASSERT(released_);
+    DCHECK(released_);
     mu_->Lock();
     released_ = false;
   }
 
   void Release() ABSL_UNLOCK_FUNCTION() {
-    GPR_DEBUG_ASSERT(!released_);
+    DCHECK(!released_);
     released_ = true;
     mu_->Unlock();
   }