chore: clean-up

Author: kgryte

lib/node_modules/@stdlib/array/fixed-endian-factory/README.md

@@ -338,27 +338,6 @@ v = arr.at( -100 );
 // returns undefined
 ```
 
-<a name="method-index-of"></a>
-
-#### TypedArrayFE.prototype.indexOf( searchElement\[, fromIndex] )
-
-Returns the first index at which a given element can be found in the typed array, or `-1` if it is not present.
-
-```javascript
-var Float64ArrayFE = fixedEndianFactory( 'float64' );
-
-var arr = new Float64ArrayFE( 'little-endian', [ 1.0, 2.0, 3.0, 4.0 ] );
-
-var idx = arr.indexOf( 2.0 );
-// returns 1
-
-idx = arr.indexOf( 5.0 );
-// returns -1
-
-idx = arr.indexOf( 3.0, 3 );
-// returns -1
-```
-
 <a name="method-every"></a>
 
 #### TypedArrayFE.prototype.every( predicate\[, thisArg] )
@@ -496,6 +475,38 @@ var v = arr.get( 100 );
 // returns undefined
 ```
 
+<a name="method-index-of"></a>
+
+#### TypedArrayFE.prototype.indexOf( searchElement\[, fromIndex] )
+
+Returns the first index at which a given element can be found.
+
+```javascript
+var Float64ArrayFE = fixedEndianFactory( 'float64' );
+
+var arr = new Float64ArrayFE( 'little-endian', [ 1.0, 2.0, 3.0, 4.0, 2.0 ] );
+
+var idx = arr.indexOf( 2.0 );
+// returns 1
+
+idx = arr.indexOf( 2.0, 2 );
+// returns 4
+
+idx = arr.indexOf( 2.0, -4 );
+// returns 1
+```
+
+If `searchElement` is not present in the array, the method returns `-1`.
+
+```javascript
+var Float64ArrayFE = fixedEndianFactory( 'float64' );
+
+var arr = new Float64ArrayFE( 'little-endian', [ 1.0, 2.0, 3.0, 4.0, 2.0 ] );
+
+var idx = arr.indexOf( 5.0 );
+// returns -1
+```
+
 <a name="method-set"></a>
 
 #### TypedArrayFE.prototype.set( arr\[, offset] )

lib/node_modules/@stdlib/array/fixed-endian-factory/lib/main.js

@@ -624,50 +624,42 @@ function factory( dtype ) { // eslint-disable-line max-lines-per-function, stdli
 	/**
 	* Returns the index of the first occurrence of a given element.
 	*
+	* @private
 	* @name indexOf
 	* @memberof TypedArray.prototype
 	* @type {Function}
 	* @param {*} searchElement - element to search for
-	* @param {integer} fromIndex - index to start the search from (optional)
+	* @param {integer} [fromIndex=0] - starting index (inclusive)
 	* @throws {TypeError} `this` must be a typed array instance
-	* @returns {integer} index of the first occurrence of the element, or -1 if not found
+	* @throws {TypeError} second argument must be an integer
+	* @returns {integer} index or -1
 	*/
 	setReadOnly( TypedArray.prototype, 'indexOf', function indexOf( searchElement, fromIndex ) {
-		var len;
 		var buf;
 		var i;
 
 		if ( !isTypedArray( this ) ) {
 			throw new TypeError( format( 'invalid invocation. `this` is not %s %s.', CHAR2ARTICLE[ dtype[0] ], CTOR_NAME ) );
 		}
-
-		len = this._length;
-
 		if ( arguments.length > 1 ) {
 			if ( !isInteger( fromIndex ) ) {
 				throw new TypeError( format( 'invalid argument. Second argument must be an integer. Value: `%s`.', fromIndex ) );
 			}
 			if ( fromIndex < 0 ) {
-				fromIndex += len;
+				fromIndex += this._length;
 				if ( fromIndex < 0 ) {
 					fromIndex = 0;
 				}
 			}
-			if ( fromIndex >= len ) {
-				return -1;
-			}
 		} else {
 			fromIndex = 0;
 		}
-
 		buf = this._buffer;
-
-		for ( i = fromIndex; i < len; i++ ) {
+		for ( i = fromIndex; i < this._length; i++ ) {
 			if ( buf[ GETTER ]( i * BYTES_PER_ELEMENT, this._isLE ) === searchElement ) {
 				return i;
 			}
 		}
-
 		return -1;
 	});
 

lib/node_modules/@stdlib/array/fixed-endian-factory/test/test.index_of.js

@@ -80,7 +80,7 @@ tape( 'the method throws an error if invoked with a `this` context which is not
 	}
 });
 
-tape( 'the method throws an error if provided `fromIndex` argument which is not an integer', function test( t ) {
+tape( 'the method throws an error if provided a second argument which is not an integer', function test( t ) {
 	var values;
 	var ctor;
 	var arr;
@@ -113,7 +113,7 @@ tape( 'the method throws an error if provided `fromIndex` argument which is not
 	}
 });
 
-tape( 'the method returns `-1` if provided fromIndex argument which exceeds array dimensions', function test( t ) {
+tape( 'the method returns `-1` if provided a second argument which exceeds array dimensions', function test( t ) {
 	var ctor;
 	var arr;
 	var v;
@@ -122,7 +122,7 @@ tape( 'the method returns `-1` if provided fromIndex argument which exceeds arra
 	ctor = factory( 'float64' );
 	arr = new ctor( 'little-endian', [ 1.0, 2.0, 3.0, 4.0, 5.0, 6.0, 7.0, 8.0, 9.0 ] );
 
-	for ( i = arr.length; i < arr.length + 5; i++ ) {
+	for ( i = arr.length; i < arr.length+5; i++ ) {
 		v = arr.indexOf( 1.0, i );
 		t.strictEqual( v, -1, 'returns expected value' );
 	}
@@ -138,12 +138,12 @@ tape( 'the method returns `-1` if operating on an empty array', function test( t
 	arr = new ctor( 'little-endian', [] );
 
 	v = arr.indexOf( 1.0 );
-	t.strictEqual( v, -1, 'returns -1 for an empty array' );
+	t.strictEqual( v, -1, 'returns expected value' );
 
 	t.end();
 });
 
-tape( 'the method returns `-1` if element not found', function test( t ) {
+tape( 'the method returns `-1` if a search element is not found', function test( t ) {
 	var ctor;
 	var arr;
 	var v;
@@ -152,12 +152,12 @@ tape( 'the method returns `-1` if element not found', function test( t ) {
 	arr = new ctor( 'little-endian', [ 1.0, 2.0, 3.0, 4.0, 5.0 ] );
 
 	v = arr.indexOf( 10.0 );
-	t.strictEqual( v, -1, 'returns -1 when element is not found' );
+	t.strictEqual( v, -1, 'returns expected value' );
 
 	t.end();
 });
 
-tape( 'the method returns the index of the first match if element is found', function test( t ) {
+tape( 'the method returns the index of the first match if an element is found', function test( t ) {
 	var ctor;
 	var arr;
 	var v;
@@ -166,7 +166,7 @@ tape( 'the method returns the index of the first match if element is found', fun
 	arr = new ctor( 'little-endian', [ 1.0, 2.0, 3.0, 4.0, 2.0 ] );
 
 	v = arr.indexOf( 2.0 );
-	t.strictEqual( v, 1, 'returns index of the first match when the element is found' );
+	t.strictEqual( v, 1, 'returns expected value' );
 
 	t.end();
 });
@@ -179,8 +179,14 @@ tape( 'the method supports specifying a starting search index', function test( t
 	ctor = factory( 'float64' );
 	arr = new ctor( 'little-endian', [ 1.0, 2.0, 3.0, 4.0, 2.0 ] );
 
+	v = arr.indexOf( 2.0, 0 );
+	t.strictEqual( v, 2, 'returns expected value' );
+
 	v = arr.indexOf( 2.0, 2 );
-	t.strictEqual( v, 4, 'returns index of the match starting from the specified index' );
+	t.strictEqual( v, 4, 'returns expected value' );
+
+	v = arr.indexOf( 1.0, 4 );
+	t.strictEqual( v, -1, 'returns expected value' );
 
 	t.end();
 });
@@ -194,7 +200,10 @@ tape( 'the method supports specifying a starting search index (negative)', funct
 	arr = new ctor( 'little-endian', [ 1.0, 2.0, 3.0, 4.0, 2.0 ] );
 
 	v = arr.indexOf( 2.0, -3 );
-	t.strictEqual( v, 4, 'returns index of the match starting from the resolved negative index' );
+	t.strictEqual( v, 4, 'returns expected value' );
+
+	v = arr.indexOf( 1.0, -3 );
+	t.strictEqual( v, -1, 'returns expected value' );
 
 	t.end();
 });
@@ -208,7 +217,10 @@ tape( 'when provided a starting index which resolves to an index less than zero,
 	arr = new ctor( 'little-endian', [ 1.0, 2.0, 3.0, 4.0, 2.0 ] );
 
 	v = arr.indexOf( 2.0, -10 );
-	t.strictEqual( v, 1, 'returns index of the match starting from the first array element when starting index resolves to less than zero' );
+	t.strictEqual( v, 1, 'returns expected value' );
+
+	v = arr.indexOf( 1.0, -10 );
+	t.strictEqual( v, 0, 'returns expected value' );
 
 	t.end();
 });