From daa2bf143433c013d29f6e767900c6a5cc5eaa30 Mon Sep 17 00:00:00 2001
From: Richard Barnes <rbarnes@meta.com>
Date: Sun, 16 Mar 2025 13:31:41 -0700
Subject: [PATCH] Fix deprecated use of 0/NULL in
 gloo/cuda_broadcast_one_to_all.h + 1

Summary:
`nullptr` is typesafe. `0` and `NULL` are not. In the future, only `nullptr` will be allowed.

This diff helps us embrace the future _now_ in service of enabling `-Wzero-as-null-pointer-constant`.

Differential Revision: D71284808
---
 gloo/cuda_broadcast_one_to_all.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/gloo/cuda_broadcast_one_to_all.h b/gloo/cuda_broadcast_one_to_all.h
index 794691221..62ce4d0ec 100644
--- a/gloo/cuda_broadcast_one_to_all.h
+++ b/gloo/cuda_broadcast_one_to_all.h
@@ -36,13 +36,13 @@ class CudaBroadcastOneToAll : public Algorithm {
   void init(
       typename std::enable_if<
           std::is_same<U, CudaHostWorkspace<T>>::value,
-          typename U::Pointer>::type* = NULL);
+          typename U::Pointer>::type* = nullptr);
 
   template <typename U = W>
   void init(
       typename std::enable_if<
           std::is_same<U, CudaDeviceWorkspace<T>>::value,
-          typename U::Pointer>::type* = NULL);
+          typename U::Pointer>::type* = nullptr);
 
   std::vector<CudaDevicePointer<T>> devicePtrs_;
   std::vector<CudaStream> streams_;