From 2e4b0ee3de941540ba7f104522313b045cdfef9c Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Iv=C3=A1n=20Perdomo?= <ivan@perdomo.me>
Date: Thu, 10 Oct 2024 09:28:22 +0200
Subject: [PATCH] IM-463 Consistent check for null

* The previous code was doing a null check after passing it to an array
builder
---
 .../integratedmodelling/klab/api/geometry/impl/NDCursor.java  | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/klab.core.api/src/main/java/org/integratedmodelling/klab/api/geometry/impl/NDCursor.java b/klab.core.api/src/main/java/org/integratedmodelling/klab/api/geometry/impl/NDCursor.java
index 72988dd2e..66a4a6d0a 100644
--- a/klab.core.api/src/main/java/org/integratedmodelling/klab/api/geometry/impl/NDCursor.java
+++ b/klab.core.api/src/main/java/org/integratedmodelling/klab/api/geometry/impl/NDCursor.java
@@ -305,8 +305,8 @@ public long defineDimensions(long... extents) {
     }
 
     public long defineDimensions(List<Long> extents) {
-        this.extents = Utils.Numbers.longArrayFromCollection(extents);
-        dimensions = extents == null ? 0 : extents.size();
+        this.dimensions = extents == null ? 0 : extents.size();
+        this.extents = extents == null? new long[0] : Utils.Numbers.longArrayFromCollection(extents);
         return initializeStrides();
     }