Skip to content

Commit 11d07cd

Browse files
Use an options argument instead of various arguments, for treeIterator()
1 parent e3a9379 commit 11d07cd

File tree

2 files changed

+6
-3
lines changed

2 files changed

+6
-3
lines changed

lib/SymbolTree.js

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -439,10 +439,13 @@ class SymbolTree {
439439
* @method treeIterator
440440
* @memberOf module:symbol-tree#
441441
* @param {Object} root
442-
* @param {Boolean} [reverse=false]
442+
* @param {Object} options
443+
* @param {Boolean} [options.reverse=false]
443444
* @return {Object} An iterable iterator (ES6)
444445
*/
445-
treeIterator(root, reverse) {
446+
treeIterator(root, options) {
447+
const reverse = options && options.reverse;
448+
446449
return new TreeIterator(
447450
this,
448451
root,

test/SymbolTree.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -996,7 +996,7 @@ test('tree iterator reverse', function(t) {
996996
tree.insertAfter(b, a);
997997

998998
const results = [];
999-
const iterator = tree.treeIterator(a, true);
999+
const iterator = tree.treeIterator(a, {reverse: true});
10001000

10011001
for (const object of iterator) {
10021002
results.push(object);

0 commit comments

Comments
 (0)