Use Array#splice directly.

This commit is contained in:
John-David Dalton
2017-04-19 12:18:37 -07:00
parent 65b148a3b2
commit 08a3f533b7
3 changed files with 4 additions and 13 deletions

View File

@@ -1,8 +1,5 @@
import assocIndexOf from './assocIndexOf.js'
/** Built-in value references. */
const splice = Array.prototype.splice
class ListCache {
/**
@@ -51,7 +48,7 @@ class ListCache {
if (index == lastIndex) {
data.pop()
} else {
splice.call(data, index, 1)
data.splice(index, 1)
}
--this.size
return true

View File

@@ -3,9 +3,6 @@ import baseIndexOf from './baseIndexOf.js'
import baseIndexOfWith from './baseIndexOfWith.js'
import copyArray from './copyArray.js'
/** Built-in value references. */
const splice = Array.prototype.splice
/**
* The base implementation of `pullAllBy`.
*
@@ -36,9 +33,9 @@ function basePullAll(array, values, iteratee, comparator) {
while ((fromIndex = indexOf(seen, computed, fromIndex, comparator)) > -1) {
if (seen !== array) {
splice.call(seen, fromIndex, 1)
seen.splice(fromIndex, 1)
}
splice.call(array, fromIndex, 1)
array.splice(fromIndex, 1)
}
}
return array

View File

@@ -1,9 +1,6 @@
import baseUnset from './baseUnset.js'
import isIndex from './isIndex.js'
/** Built-in value references. */
const splice = Array.prototype.splice
/**
* The base implementation of `pullAt` without support for individual
* indexes or capturing the removed elements.
@@ -23,7 +20,7 @@ function basePullAt(array, indexes) {
if (length == lastIndex || index !== previous) {
previous = index
if (isIndex(index)) {
splice.call(array, index, 1)
array.splice(index, 1)
} else {
baseUnset(array, index)
}