modify initialization of 'ndim' in SymEntry initializers s.t. it's va…

…lue isn't assigned directly from a param formal (Bears-R-Us#2930) Signed-off-by: Jeremiah Corrado <jeremiah.corrado@hpe.com>
stoutes · Jan 25, 2024 · 8a7e9c0 · 8a7e9c0
1 parent eded4aa
commit 8a7e9c0
Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/src/compat/e-132/ArkoudaSymEntryCompat.chpl b/src/compat/e-132/ArkoudaSymEntryCompat.chpl
@@ -67,6 +67,7 @@ module ArkoudaSymEntryCompat {
     this.a = try! makeDistArray((...args), etype);
     init this;
     this.shape = tupShapeString(this.tupShape);
+    this.ndim = this.tupShape.size;
   }
 
   /*
@@ -87,5 +88,6 @@ module ArkoudaSymEntryCompat {
     this.max_bits=max_bits;
     init this;
     this.shape = tupShapeString(this.tupShape);
+    this.ndim = this.tupShape.size;
   }
 }
diff --git a/src/compat/eq-131/ArkoudaSymEntryCompat.chpl b/src/compat/eq-131/ArkoudaSymEntryCompat.chpl
@@ -61,7 +61,7 @@ module ArkoudaSymEntryCompat {
     this.a = try! makeDistArray((...args), etype);
     this.complete();
     this.shape = tupShapeString(this.tupShape);
-    this.ndim = N;
+    this.ndim = this.tupShape.size;
   }
 
   /*
@@ -82,6 +82,6 @@ module ArkoudaSymEntryCompat {
     this.max_bits=max_bits;
     this.complete();
     this.shape = tupShapeString(this.tupShape);
-    this.ndim = D.rank;
+    this.ndim = this.tupShape.size;
   }
 }
diff --git a/src/compat/gt-132/ArkoudaSymEntryCompat.chpl b/src/compat/gt-132/ArkoudaSymEntryCompat.chpl
@@ -67,8 +67,7 @@ module ArkoudaSymEntryCompat {
     this.a = try! makeDistArray((...args), etype);
     init this;
     this.shape = tupShapeString(this.tupShape);
-    // commented out for perf reasons, needs further investigation
-    // this.ndim = N;
+    this.ndim = this.tupShape.size;
   }
 
   /*
@@ -89,7 +88,6 @@ module ArkoudaSymEntryCompat {
     this.max_bits=max_bits;
     init this;
     this.shape = tupShapeString(this.tupShape);
-    // commented out for perf reasons, needs further investigation
-    // this.ndim = D.rank;
+    this.ndim = this.tupShape.size;
   }
 }