Thanks for that!
That made it obvious that we were never actually freeing the buffers allocated on a BufferStream when it is destroyed.
I thought that (a) the client side did that, and (b) the server side *also* did that.
But it turns out that stream->associate_buffer() is actually a bunch of entirely unused bookeeping, so... :)
« Back to merge proposal
Thanks for that!
That made it obvious that we were never actually freeing the buffers allocated on a BufferStream when it is destroyed.
I thought that (a) the client side did that, and (b) the server side *also* did that.
But it turns out that stream- >associate_ buffer( ) is actually a bunch of entirely unused bookeeping, so... :)