Performance issues with large numbers of pipes about pipes-rs HOT 2 CLOSED

I’ve improved memory usage in #57 and #58.

Locally I tried to reuse the same allocation for all pipes rather than creating a new one each reset, but ran into some unsolvable borrow checker errors that have led to me giving up on the idea.

(To elaborate on that: there is a loop through &mut pipes that mutates self inside it. This is fine at the moment because pipes is a local variable; if pipes is moved to a field on self then you get an error about two mutable references being held to self at once (one in the for pipe in &mut self.pipes and the other in the loop body). This is reasonable and expected if you consider the code, but is too annoying to work around just to avoid a single allocation each reset, something which happens rarely.)

from pipes-rs.

Seeing as all my original suggestions to improve performance have been implemented and performance isn’t really an issue to begin with (as long as you don’t use insane numbers of pipes!), I’m closing this issue.

from pipes-rs.