draft: synchronousTransformIterator

[jeswr]

As discussed in #75 (comment).

TODOs

• Add more coverage
• See if

  AsyncIterator/asynciterator.ts

  Lines 905 to 911 in 3a5b62c

  	if (!this.root.readable) {
  	this.readable = false;
  	// TODO: See if this should be here
  	if (this.root.done)
  	this.close();
  	return null;
  	}

  is necessary. @RubenVerborgh the question here is - can we just assume the root is .readable if the parent source is .readable. If we assume that all of the intermediary sources are SynchronousTransformIterators as is the case when doing chaining then this will be true; but I am not sure if we should be making this assumption.

Quick testing indicates that this results in a ~2x performance increase on the tests in #75

[jacoscaz]

Should this be a part of the changes summarized at #44 (comment) ?

[jacoscaz]

@jeswr is this something you're still working on? Would this be releasable as 3.x or 4.x?

[jeswr]

@jacoscaz The changes are still valid - and there are no breaking changes. I can try and work on the test coverage over the weekend depending on my time (though I'm looking a bit stretched thin at this point).

[jacoscaz]

No pressure or rush meant! Just checking, I was having a look at updating my summary in #44 and wasn't sure about this one. Nice!