Remove check for type undefined, which is now redundant.

Cristiano Calcagno · Cristiano Calcagno · commit 13e8fa57f467 · 2018-05-07T10:18:22.000+02:00
diff --git a/jscomp/runtime/caml_obj.ml b/jscomp/runtime/caml_obj.ml
@@ -185,7 +185,6 @@ let rec caml_compare (a : Obj.t) (b : Obj.t) : int =
     | false, true -> 1 
     | false, false -> 
       if a_type = "boolean"
-      || a_type = "undefined"
       then (* TODO: refine semantics when comparing with [null] *)
         unsafe_js_compare a b
       else if a_type = "function" || b_type = "function"
diff --git a/lib/js/caml_obj.js b/lib/js/caml_obj.js
@@ -90,7 +90,7 @@ function caml_compare(_a, _b) {
           }
         } else if (is_b_number) {
           return 1;
-        } else if (a_type === "boolean" || a_type === "undefined") {
+        } else if (a_type === "boolean") {
           var x = a;
           var y = b;
           if (x === y) {

Original file line number	Diff line number	Diff line change
`@@ -90,7 +90,7 @@ function caml_compare(_a, _b) {`
`90`	`90`	`}`
`91`	`91`	`} else if (is_b_number) {`
`92`	`92`	`return 1;`
`93`		`- } else if (a_type === "boolean" \|\| a_type === "undefined") {`
	`93`	`+ } else if (a_type === "boolean") {`
`94`	`94`	`var x = a;`
`95`	`95`	`var y = b;`
`96`	`96`	`if (x === y) {`