Skip to content

Commit a79e000

Browse files
committed
Refactor to ignore iterator return values
1 parent b79f3d6 commit a79e000

File tree

2 files changed

+1
-81
lines changed

2 files changed

+1
-81
lines changed

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

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,6 @@
2323
var setReadOnly = require( '@stdlib/utils/define-nonenumerable-read-only-property' );
2424
var isFunction = require( '@stdlib/assert/is-function' );
2525
var isIteratorLike = require( '@stdlib/assert/is-iterator-like' );
26-
var hasOwnProp = require( '@stdlib/assert/has-own-property' );
2726
var iteratorSymbol = require( '@stdlib/symbol/iterator' );
2827

2928

@@ -89,7 +88,6 @@ function iterMap( iterator, fcn, thisArg ) {
8988
* @returns {Object} iterator protocol-compliant object
9089
*/
9190
function next() {
92-
var out;
9391
var v;
9492
if ( FLG ) {
9593
return {
@@ -100,12 +98,7 @@ function iterMap( iterator, fcn, thisArg ) {
10098
i += 1;
10199
if ( v.done ) {
102100
FLG = true;
103-
out = {};
104-
if ( hasOwnProp( v, 'value' ) ) {
105-
out.value = fcn.call( thisArg, v.value, i );
106-
}
107-
out.done = true;
108-
return out;
101+
return v;
109102
}
110103
return {
111104
'value': fcn.call( thisArg, v.value, i ),

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

Lines changed: 0 additions & 73 deletions
Original file line numberDiff line numberDiff line change
@@ -28,37 +28,6 @@ var noop = require( '@stdlib/utils/noop' );
2828
var iterMap = require( './../lib' );
2929

3030

31-
// FUNCTIONS //
32-
33-
function createIterator( arr ) {
34-
var len;
35-
var it;
36-
var i;
37-
38-
len = arr.length;
39-
i = -1;
40-
41-
it = {};
42-
it.next = next;
43-
44-
return it;
45-
46-
function next() {
47-
var out;
48-
i += 1;
49-
if ( i < len ) {
50-
out = {};
51-
out.value = arr[ i ];
52-
out.done = ( i === len-1 );
53-
return out;
54-
}
55-
return {
56-
'done': true
57-
};
58-
}
59-
}
60-
61-
6231
// TESTS //
6332

6433
tape( 'main export is a function', function test( t ) {
@@ -144,48 +113,6 @@ tape( 'the function returns an iterator protocol-compliant object', function tes
144113
}
145114
});
146115

147-
tape( 'the function returns an iterator protocol-compliant object (value+done)', function test( t ) {
148-
var expected;
149-
var values;
150-
var actual;
151-
var it;
152-
var i;
153-
154-
values = [ 1, 2, 3, 4 ];
155-
expected = [
156-
{
157-
'value': 1,
158-
'done': false
159-
},
160-
{
161-
'value': 4,
162-
'done': false
163-
},
164-
{
165-
'value': 9,
166-
'done': false
167-
},
168-
{
169-
'value': 16,
170-
'done': true
171-
}
172-
];
173-
174-
it = iterMap( createIterator( values ), scale );
175-
t.equal( it.next.length, 0, 'has zero arity' );
176-
177-
actual = [];
178-
for ( i = 0; i < values.length; i++ ) {
179-
actual.push( it.next() );
180-
}
181-
t.deepEqual( actual, expected, 'returns expected values' );
182-
t.end();
183-
184-
function scale( v, i ) {
185-
return v * (i+1);
186-
}
187-
});
188-
189116
tape( 'the function returns an iterator protocol-compliant object which invokes a provided function for each iterated value', function test( t ) {
190117
var expected;
191118
var opts;

0 commit comments

Comments
 (0)