potential performance issue in List about tui-rs

[cardoe]

I should mention that Iterator.zip() doesn't work if the style iterator has less items than the data iterator because zip() stops when any of the iterators run out of data. So people would be confused why their data was being truncated.

[fdehau]

Hi, there are several valid points here. First of all, there are certainly a lot of places in the code where there is room for improvements regarding both the API exposed to the caller and the performance.

I took the liberty to implement your ideas in a dedicated branch #11. The major change required for this to work is to update the Widget signature. Indeed, until now the draw call was done using a simple & to the widget. Consuming the iterator inside the draw call would then require &mut if I'm not mistaken. This change the behavior of the widget that should now be seen as a "consumable" object. However, this does not appear to me as problem since it was one of the goals of the library anyway.

I would appreciate your feedback on this if you have some time :)

[cardoe]

Looks good. Sorry for the delay in looking at it. There were some other changes to the API so that delayed me a bit from trying it out. I like it.