remove internal usage of %assert, since ocaml exception already produces good stacktrace

Author: bobzhang

Diff for: jscomp/others/belt_Array.ml

@@ -22,13 +22,13 @@ external getUndefined: 'a array -> int -> 'a Js.undefined = "%array_unsafe_get"
 let get arr i =
   if i >= 0 && i < length arr then Some (getUnsafe arr i) else None
 let getExn arr i =
-    [%assert i >= 0 && i < length arr] ;
+    assert (i >= 0 && i < length arr) ;
     getUnsafe arr i
 let set arr i v =
   if i >= 0 && i < length arr then (setUnsafe arr i v; true) else false
 
 let setExn arr i v =
-  [%assert i >= 0 && i < length arr];
+  assert (i >= 0 && i < length arr);
   setUnsafe arr i v
 
 

Diff for: jscomp/others/belt_internalAVLset.ml

@@ -287,7 +287,7 @@ let rec checkInvariantInternal (v : _ t) =
   | Some n ->
     let {left = l; right = r} = n   in
     let diff = height l - height r  in
-    [%assert diff <=2 && diff >= -2];
+    assert (diff <=2 && diff >= -2);
     checkInvariantInternal l;
     checkInvariantInternal r
 

Diff for: jscomp/others/belt_internalAVLtree.ml

@@ -370,7 +370,7 @@ let rec checkInvariantInternal (v : _ t) =
   | Some n ->
     let l,r = n.left , n.right in
     let diff = treeHeight l - treeHeight r  in
-    [%assert diff <=2 && diff >= -2 ];
+    assert (diff <=2 && diff >= -2 );
     checkInvariantInternal l;
     checkInvariantInternal r