proposal: x/text/transform: Add func (*Reader).Reset(r io.Reader, t Transformer)

[icholy]

The x/test/transform.Reader constructor allocates two 4kb buffers. I want to reuse existing Readers with a sync.Pool to reduce allocations. I'm proposing a Reset method similar to bufio.Reader.Reset

func (rd *Reader) Reset(r io.Reader, t Transformer) {
	t.Reset()
	rd.r = r
	rd.t = t
	rd.err = nil
	rd.dst0 = 0
	rd.dst1 = 0
	rd.src0 = 0
	rd.src1 = 0
}

[ianlancetaylor]

CC @mpvl

[icholy]

@ianlancetaylor is @mpvl still actively participating in this project? I appreciate that people are busy, but #49117 has been sitting for over a year. I'm wondering if we're waiting for someone who's never going to respond.

[ianlancetaylor]

@mpvl still responds from time to time. But in any case this proposal is still in the incoming queue along with a couple of hundred other proposals. We aren't waiting for @mpvl, we're waiting for the proposal review committee.

[icholy]

@ianlancetaylor any chance to look at this?

[ianlancetaylor]

I'll make sure it is in the queue.