Remove Number.isNaN IE polyfill

This commit is contained in:
adroitwhiz 2019-04-25 16:41:51 -04:00
parent 0f592519d6
commit 3410f42e5e

View file

@ -1,13 +1,5 @@
const Color = require('../util/color'); const Color = require('../util/color');
/**
* Store and possibly polyfill Number.isNaN. Number.isNaN can save time over
* self.isNaN by not coercing its input. We need to polyfill it to support
* Internet Explorer.
* @const
*/
const _NumberIsNaN = Number.isNaN || isNaN;
/** /**
* @fileoverview * @fileoverview
* Utilities for casting and comparing Scratch data-types. * Utilities for casting and comparing Scratch data-types.
@ -33,13 +25,13 @@ class Cast {
if (typeof value === 'number') { if (typeof value === 'number') {
// Scratch treats NaN as 0, when needed as a number. // Scratch treats NaN as 0, when needed as a number.
// E.g., 0 + NaN -> 0. // E.g., 0 + NaN -> 0.
if (_NumberIsNaN(value)) { if (Number.isNaN(value)) {
return 0; return 0;
} }
return value; return value;
} }
const n = Number(value); const n = Number(value);
if (_NumberIsNaN(n)) { if (Number.isNaN(n)) {
// Scratch treats NaN as 0, when needed as a number. // Scratch treats NaN as 0, when needed as a number.
// E.g., 0 + NaN -> 0. // E.g., 0 + NaN -> 0.
return 0; return 0;