Clarify comments on _.support.spliceObjects and _.isFunction. [ci skip]

This commit is contained in:
John-David Dalton
2014-11-09 09:52:24 -08:00
parent 0c54fac651
commit 65e30c13e2

View File

@@ -1177,12 +1177,11 @@
* Detect if `Array#shift` and `Array#splice` augment array-like objects * Detect if `Array#shift` and `Array#splice` augment array-like objects
* correctly. * correctly.
* *
* Firefox < 10, IE compatibility mode, and IE < 9 have buggy Array `shift()` * Firefox < 10, compatibility modes of IE 8, and IE < 9 have buggy Array `shift()`
* and `splice()` functions that fail to remove the last element, `value[0]`, * and `splice()` functions that fail to remove the last element, `value[0]`,
* of array-like objects even though the `length` property is set to `0`. * of array-like objects even though the `length` property is set to `0`.
* The `shift()` method is buggy in IE 8 compatibility mode, while `splice()` * The `shift()` method is buggy in compatibility modes of IE 8, while `splice()`
* is buggy regardless of mode in IE < 9 and buggy in compatibility mode * is buggy regardless of mode in IE < 9.
* in IE 9.
* *
* @memberOf _.support * @memberOf _.support
* @type boolean * @type boolean
@@ -7373,7 +7372,7 @@
* // => false * // => false
*/ */
function isFunction(value) { function isFunction(value) {
// avoid a Chakra bug in IE 11 // use `|| false` to avoid a Chakra bug in compatibility modes of IE 11
// https://github.com/jashkenas/underscore/issues/1621 // https://github.com/jashkenas/underscore/issues/1621
return typeof value == 'function' || false; return typeof value == 'function' || false;
} }