feat: use panic where appropriate

Author: glennsl

src/Core__List.mjs

@@ -4,6 +4,7 @@ import * as Curry from "rescript/lib/es6/curry.js";
 import * as Belt_Array from "rescript/lib/es6/belt_Array.js";
 import * as Caml_option from "rescript/lib/es6/caml_option.js";
 import * as Core__Array from "./Core__Array.mjs";
+import * as Core__Error from "./Core__Error.mjs";
 
 function head(x) {
   if (x) {
@@ -15,11 +16,9 @@ function head(x) {
 function headExn(x) {
   if (x) {
     return x.hd;
+  } else {
+    return Core__Error.panic("List.headExn: list is empty");
   }
-  throw {
-        RE_EXN_ID: "Not_found",
-        Error: new Error()
-      };
 }
 
 function tail(x) {
@@ -32,11 +31,9 @@ function tail(x) {
 function tailExn(x) {
   if (x) {
     return x.tl;
+  } else {
+    return Core__Error.panic("List.tailExn: list is empty");
   }
-  throw {
-        RE_EXN_ID: "Not_found",
-        Error: new Error()
-      };
 }
 
 function add(xs, x) {
@@ -70,29 +67,24 @@ function get(x, n) {
 
 function getExn(x, n) {
   if (n < 0) {
-    throw {
-          RE_EXN_ID: "Not_found",
-          Error: new Error()
-        };
-  }
-  var _x = x;
-  var _n = n;
-  while(true) {
-    var n$1 = _n;
-    var x$1 = _x;
-    if (x$1) {
+    return Core__Error.panic("List.getExn: n < 0");
+  } else {
+    var _x = x;
+    var _n = n;
+    while(true) {
+      var n$1 = _n;
+      var x$1 = _x;
+      if (!x$1) {
+        return Core__Error.panic("List.nthAuxAssert: list is empty");
+      }
       if (n$1 === 0) {
         return x$1.hd;
       }
       _n = n$1 - 1 | 0;
       _x = x$1.tl;
       continue ;
-    }
-    throw {
-          RE_EXN_ID: "Not_found",
-          Error: new Error()
-        };
-  };
+    };
+  }
 }
 
 function partitionAux(p, _cell, _precX, _precY) {

src/Core__List.res

@@ -94,7 +94,7 @@ let head = x =>
 
 let headExn = x =>
   switch x {
-  | list{} => raise(Not_found)
+  | list{} => Core__Error.panic("List.headExn: list is empty")
   | list{x, ..._} => x
   }
 
@@ -106,7 +106,7 @@ let tail = x =>
 
 let tailExn = x =>
   switch x {
-  | list{} => raise(Not_found)
+  | list{} => Core__Error.panic("List.tailExn: list is empty")
   | list{_, ...t} => t
   }
 
@@ -132,7 +132,7 @@ let rec nthAuxAssert = (x, n) =>
     } else {
       nthAuxAssert(t, n - 1)
     }
-  | _ => raise(Not_found)
+  | list{} => Core__Error.panic("List.nthAuxAssert: list is empty")
   }
 
 let get = (x, n) =>
@@ -144,7 +144,7 @@ let get = (x, n) =>
 
 let getExn = (x, n) =>
   if n < 0 {
-    raise(Not_found)
+    Core__Error.panic("List.getExn: n < 0")
   } else {
     nthAuxAssert(x, n)
   }

src/Core__List.resi

@@ -82,7 +82,7 @@ List.headExn(list{}) // Raises an Error
 
 ## Exceptions
 
-- Raises an Error if list is empty.
+- Panics if list is empty.
 
 */
 let headExn: t<'a> => 'a
@@ -114,7 +114,7 @@ List.tailExn(list{}) // Raises an Error
 
 ## Exceptions
 
-- Raises an Error if list is empty.
+- Panics if list is empty.
 */
 let tailExn: t<'a> => t<'a>
 
@@ -162,7 +162,7 @@ abc->List.getExn(4) // Raises an Error
 
 ## Exceptions
 
-- Raises an Error if `index` is larger than the length of list.
+- Panics if `index` is less than 0 or larger than the length of list.
 */
 let getExn: (t<'a>, int) => 'a
 

src/Core__Option.mjs

@@ -2,6 +2,7 @@
 
 import * as Curry from "rescript/lib/es6/curry.js";
 import * as Caml_option from "rescript/lib/es6/caml_option.js";
+import * as Core__Error from "./Core__Error.mjs";
 
 function filter(opt, p) {
   var p$1 = Curry.__1(p);
@@ -22,11 +23,9 @@ function forEach(opt, f) {
 function getExn(x) {
   if (x !== undefined) {
     return Caml_option.valFromOption(x);
+  } else {
+    return Core__Error.panic("List.headExn: option is None");
   }
-  throw {
-        RE_EXN_ID: "Not_found",
-        Error: new Error()
-      };
 }
 
 function mapWithDefault(opt, $$default, f) {

src/Core__Option.res

@@ -41,7 +41,7 @@ let forEach = (opt, f) => forEachU(opt, (. x) => f(x))
 let getExn = x =>
   switch x {
   | Some(x) => x
-  | None => raise(Not_found)
+  | None => Core__Error.panic("List.headExn: option is None")
   }
 
 external getUnsafe: option<'a> => 'a = "%identity"

src/Core__Option.resi

@@ -75,7 +75,7 @@ Option.getExn(None) /* Raises an Error */
 
 ## Exceptions
 
-- Raises an error if `opt` is `None`
+- Panics if `opt` is `None`
 */
 let getExn: option<'a> => 'a
 

src/Core__Result.mjs

@@ -1,15 +1,14 @@
 // Generated by ReScript, PLEASE EDIT WITH CARE
 
 import * as Curry from "rescript/lib/es6/curry.js";
+import * as Core__Error from "./Core__Error.mjs";
 
 function getExn(x) {
   if (x.TAG === /* Ok */0) {
     return x._0;
+  } else {
+    return Core__Error.panic("Result.getExn: result is Error");
   }
-  throw {
-        RE_EXN_ID: "Not_found",
-        Error: new Error()
-      };
 }
 
 function mapWithDefault(opt, $$default, f) {

src/Core__Result.res

@@ -27,7 +27,7 @@ type t<'a, 'b> = result<'a, 'b> = Ok('a) | Error('b)
 let getExn = x =>
   switch x {
   | Ok(x) => x
-  | Error(_) => raise(Not_found)
+  | Error(_) => Core__Error.panic("Result.getExn: result is Error")
   }
 
 let mapWithDefaultU = (opt, default, f) =>

src/Core__Result.resi

@@ -55,6 +55,10 @@ type t<'a, 'b> = result<'a, 'b> = Ok('a) | Error('b)
 
   Result.getExn(Result.Error("Invalid data")) /* raises exception */
   ```
+
+## Exceptions
+
+- Panics if `res` is `Error(_)`
 */
 let getExn: t<'a, 'b> => 'a