Fixed mistake in "Array.reverse"

Fixed a mistake in the definition of "Array.reverse" that caused the function to unexpectedly panic when given an array with length 0.
schwalbe-t · Jan 24, 2023 · 21fa3c1 · 21fa3c1
1 parent 2f515c2
commit 21fa3c1
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/currant.js b/currant.js
@@ -2713,7 +2713,7 @@ const CURRANT_STD_ARRAYS = `
         itemType: fun = f@currantArrayItemType;
 
         reverse: fun = (src: arr) -> arr {
-            if(len(src) == 0u64, <- { -> [itemType(src): 0u64, 0u8]; });
+            if(len(src) == 0u64, <- { -> [itemType(src): 0u64: none]; });
             dest: arr = [itemType(src): len(src): src[0u64]];
             for(range(0u64, len(src)), (i: u64) -> lpa {
                 dest[len(src) - 1u64 - i] = src[i];

diff --git a/currant/defaults/arrays.js b/currant/defaults/arrays.js
@@ -6,7 +6,7 @@ const CURRANT_STD_ARRAYS = `
         itemType: fun = f@currantArrayItemType;
 
         reverse: fun = (src: arr) -> arr {
-            if(len(src) == 0u64, <- { -> [itemType(src): 0u64, 0u8]; });
+            if(len(src) == 0u64, <- { -> [itemType(src): 0u64: none]; });
             dest: arr = [itemType(src): len(src): src[0u64]];
             for(range(0u64, len(src)), (i: u64) -> lpa {
                 dest[len(src) - 1u64 - i] = src[i];