Add default member initializers to fix cppcheck 2.21 warnings

* Add default member initializers to fix cppcheck 2.21 warnings

cppcheck 2.21.0 began reporting uninitMemberVarNoCtor for struct members
without default initializers under --enable=warning, failing CI. Add {}
or explicit zero defaults to the affected members in AllocRecord,
ScopeKey, APICBranchBody, CuBQLNode, CudaTimingRange, HashGrid_t,
noise_level_t, SolidAngleProps, and tile_coord_iter_t.

Signed-off-by: Nicolas Capens <ncapens@nvidia.com>

Approved-by: Eric Shi <ershi@nvidia.com>

See merge request omniverse/warp!2486

Author: c0d1f1ed

warp/native/alloc_tracker.h

@@ -17,10 +17,10 @@ enum AllocKind {
 };
 
 struct AllocRecord {
-    size_t size;
-    size_t seq;  // allocation sequence number (monotonically increasing)
-    AllocKind kind;
-    int device_ordinal;  // -1 for host/pinned
+    size_t size = 0;
+    size_t seq = 0;  // allocation sequence number (monotonically increasing)
+    AllocKind kind = {};
+    int device_ordinal = 0;  // -1 for host/pinned
     std::string tag;  // Python call-site info or "(native)"
     std::string scope;  // e.g. "simulation/collision"
 };
@@ -78,8 +78,8 @@ class AllocTracker {
 
     struct ScopeKey {
         std::string scope;
-        AllocKind kind;
-        int ordinal;  // only meaningful for ALLOC_KIND_DEVICE
+        AllocKind kind = {};
+        int ordinal = 0;  // only meaningful for ALLOC_KIND_DEVICE
 
         bool operator==(const ScopeKey& o) const { return scope == o.scope && kind == o.kind && ordinal == o.ordinal; }
     };

warp/native/apic.cpp

@@ -360,7 +360,7 @@ struct APICBranchStart {
 
 struct APICBranchBody {
     std::vector<uint8_t> data;
-    uint32_t op_count;
+    uint32_t op_count = 0;
 };
 
 APICBranchStart* wp_apic_begin_branch(APICState* state)

warp/native/bvh.h

@@ -213,7 +213,7 @@ struct BVH {
 struct CuBQLNode {
     vec3 lower;
     vec3 upper;
-    uint64_t admin;
+    uint64_t admin = 0;
 };
 
 struct CuBQLBVH {

warp/native/cuda_util.h

@@ -299,11 +299,11 @@ class ContextGuard {
 
 // CUDA timing range used during event-based timing
 struct CudaTimingRange {
-    void* context;
-    const char* name;
-    int flag;
-    CUevent start;
-    CUevent end;
+    void* context = nullptr;
+    const char* name = nullptr;
+    int flag = 0;
+    CUevent start = {};
+    CUevent end = {};
 };
 
 // Timing result used to pass timings to Python

warp/native/hashgrid.h

@@ -9,24 +9,24 @@ namespace wp {
 // point_ids is at a consistent offset regardless of Type. This allows
 // hash_grid_point_id to work without knowing the grid's scalar type.
 template <typename Type> struct HashGrid_t {
-    int* point_cells { nullptr };  // cell id of a point
-    int* point_ids { nullptr };  // index to original point
+    int* point_cells = nullptr;  // cell id of a point
+    int* point_ids = nullptr;  // index to original point
 
-    int* cell_starts { nullptr };  // start index of a range of indices belonging to a cell, dim_x*dim_y*dim_z in length
-    int* cell_ends { nullptr };  // end index of a range of indices belonging to a cell, dim_x*dim_y*dim_z in length
+    int* cell_starts = nullptr;  // start index of a range of indices belonging to a cell, dim_x*dim_y*dim_z in length
+    int* cell_ends = nullptr;  // end index of a range of indices belonging to a cell, dim_x*dim_y*dim_z in length
 
-    int dim_x;
-    int dim_y;
-    int dim_z;
+    int dim_x = 0;
+    int dim_y = 0;
+    int dim_z = 0;
 
-    int num_points;
-    int max_points;
+    int num_points = 0;
+    int max_points = 0;
 
-    void* context { nullptr };
+    void* context = nullptr;
 
     // Type-dependent fields at end (different sizes for half/float/double)
-    Type cell_width;
-    Type cell_width_inv;
+    Type cell_width = {};
+    Type cell_width_inv = {};
 };
 
 // Type aliases for backward compatibility and convenience

warp/native/noise.h

@@ -457,7 +457,7 @@ inline CUDA_CALLABLE vec4 noise_4d_gradient(
 // (wrt all input axes), and Hessian (symmetric). Kept as a templated
 // struct so the composition rule is shared across 2D / 3D / 4D.
 template <unsigned N> struct noise_level_t {
-    float val;
+    float val = 0.0f;
     vec_t<N, float> grad;
     mat_t<N, N, float> hess;
 };

warp/native/solid_angle.h

@@ -36,25 +36,25 @@ class SolidAngleProps {
     vec3 n_ij_diag;
     vec3 n_ijk_diag;
 
-    float sum_permute_n_xyz;
-    float two_n_xxy_n_yxx;
-    float two_n_xxz_n_zxx;
-    float two_n_yyz_n_zyy;
-    float two_n_yyx_n_xyy;
-    float two_n_zzx_n_xzz;
-    float two_n_zzy_n_yzz;
-
-    float n_xy;
-    float n_yx;
-    float n_yz;
-    float n_zy;
-    float n_zx;
-    float n_xz;
+    float sum_permute_n_xyz = 0.0f;
+    float two_n_xxy_n_yxx = 0.0f;
+    float two_n_xxz_n_zxx = 0.0f;
+    float two_n_yyz_n_zyy = 0.0f;
+    float two_n_yyx_n_xyy = 0.0f;
+    float two_n_zzx_n_xzz = 0.0f;
+    float two_n_zzy_n_yzz = 0.0f;
+
+    float n_xy = 0.0f;
+    float n_yx = 0.0f;
+    float n_yz = 0.0f;
+    float n_zy = 0.0f;
+    float n_zx = 0.0f;
+    float n_xz = 0.0f;
 
     bounds3 box;
     vec3 area_P;
-    float area;
-    float max_p_dist_sq;
+    float area = 0.0f;
+    float max_p_dist_sq = 0.0f;
 };
 
 CUDA_CALLABLE inline void compute_integrals(

warp/native/tile.h

@@ -624,8 +624,8 @@ template <typename Shape> struct tile_coord_iter_t {
     static constexpr int lastdim = N - 1;
 
     tile_coord_t<N> coord;
-    int strides[N];  // byte strides; bounded by array_t::strides[] int32 ABI
-    int64_t byte_offset;  // cumulative bytes; can exceed 2 GiB for large arrays
+    int strides[N] = {};  // byte strides; bounded by array_t::strides[] int32 ABI
+    int64_t byte_offset = 0;  // cumulative bytes; can exceed 2 GiB for large arrays
 
     // initialize from a starting coordinate and the global array byte strides/offsets
     inline CUDA_CALLABLE void init(const tile_coord_t<N>& c, const int* byte_strides, const int* tile_offset)