Refactor to ignore iterator return values

Author: kgryte

lib/node_modules/@stdlib/iter/every/benchmark/benchmark.js

@@ -38,23 +38,27 @@ function createIterator( arr ) {
 
 	it = {};
 	it.next = next;
+	it.reset = reset;
 
 	return it;
 
 	function next() {
 		i += 1;
-		if ( i < len-1 ) {
+		if ( i < len ) {
 			return {
 				'value': arr[ i ],
 				'done': false
 			};
 		}
 		i = -1; // reset index
 		return {
-			'value': arr[ len-1 ],
 			'done': true
 		};
 	}
+
+	function reset() {
+		i = -1;
+	}
 }
 
 
@@ -73,6 +77,7 @@ bench( pkg, function benchmark( b ) {
 		if ( !isBoolean( bool ) ) {
 			b.fail( 'should return a boolean' );
 		}
+		arr.reset();
 	}
 	b.toc();
 	if ( !isBoolean( bool ) ) {
@@ -104,6 +109,7 @@ bench( pkg+'::loop', function benchmark( b ) {
 		if ( !isBoolean( bool ) ) {
 			b.fail( 'should be a boolean' );
 		}
+		arr.reset();
 	}
 	b.toc();
 	if ( !isBoolean( bool ) ) {

lib/node_modules/@stdlib/iter/every/lib/main.js

@@ -21,7 +21,6 @@
 // MODULES //
 
 var isIteratorLike = require( '@stdlib/assert/is-iterator-like' );
-var hasOwnProp = require( '@stdlib/assert/has-own-property' );
 
 
 // MAIN //
@@ -48,20 +47,13 @@ function iterEvery( iterator ) {
 	}
 	while ( true ) {
 		v = iterator.next();
-		if ( v.value ) {
-			if ( v.done ) {
-				break;
-			}
-		} else if ( v.done ) {
-			if ( hasOwnProp( v, 'value' ) ) {
-				return false;
-			}
-			break;
-		} else {
+		if ( v.done ) {
+			return true;
+		}
+		if ( !v.value ) {
 			return false;
 		}
 	}
-	return true;
 }
 
 

lib/node_modules/@stdlib/iter/every/test/test.js

@@ -84,50 +84,6 @@ tape( 'the function returns `true` if all iterated values are truthy', function
 	t.end();
 });
 
-tape( 'the function returns `true` if all iterated values are truthy (value+done)', function test( t ) {
-	var bool;
-	var arr;
-
-	arr = createIterator( [ 1, 1, 1, 1 ] );
-	bool = iterEvery( arr );
-
-	t.strictEqual( bool, true, 'returns true' );
-	t.end();
-
-	function createIterator( arr ) {
-		var len;
-		var it;
-		var i;
-
-		len = arr.length;
-		i = -1;
-
-		it = {};
-		it.next = next;
-
-		return it;
-
-		function next() {
-			i += 1;
-			if ( i < len-1 ) {
-				return {
-					'value': arr[ i ],
-					'done': false
-				};
-			}
-			if ( i === len-1 ) {
-				return {
-					'value': arr[ i ],
-					'done': true
-				};
-			}
-			return {
-				'done': true
-			};
-		}
-	}
-});
-
 tape( 'the function returns `false` if at least one iterated value is falsy', function test( t ) {
 	var bool;
 	var arr;
@@ -138,47 +94,3 @@ tape( 'the function returns `false` if at least one iterated value is falsy', fu
 	t.strictEqual( bool, false, 'returns false' );
 	t.end();
 });
-
-tape( 'the function returns `false` if at least one iterated value is falsy (value+done)', function test( t ) {
-	var bool;
-	var arr;
-
-	arr = createIterator( [ 1, 1, 1, 0 ] );
-	bool = iterEvery( arr );
-
-	t.strictEqual( bool, false, 'returns false' );
-	t.end();
-
-	function createIterator( arr ) {
-		var len;
-		var it;
-		var i;
-
-		len = arr.length;
-		i = -1;
-
-		it = {};
-		it.next = next;
-
-		return it;
-
-		function next() {
-			i += 1;
-			if ( i < len-1 ) {
-				return {
-					'value': arr[ i ],
-					'done': false
-				};
-			}
-			if ( i === len-1 ) {
-				return {
-					'value': arr[ i ],
-					'done': true
-				};
-			}
-			return {
-				'done': true
-			};
-		}
-	}
-});