From: Patrick Steinhardt <ps@pks.im>
Date: Fri, 21 Nov 2025 06:32:00 GMT
Subject: Re: [PATCH 02/18] streaming: drop the `open()` callback function
Message-ID: <aSAHYNBCMwYsFMYM@pks.im>
In-Reply-To: <g74hupkwedtclb3gxomhxj6w4rqqzn3tsostdriauvn3gu2cw2@wxgwulitxbtq>

On Wed, Nov 19, 2025 at 01:01:03PM -0600, Justin Tobler wrote:
> On 25/11/19 08:47AM, Patrick Steinhardt wrote:
> > Instead, drop the callback entirely and refactor `istream_source()` so
> > that we open the streams immediately. This unblocks a subsequent step,
> > where we'll also start to allocate the structure in the source-specific
> > logic.
> 
> Out of curiousity, is there any reason we would ever want to delay
> opening the source read stream? If not, then I agree it makes more sense
> to just open the stream at time of its initialization.

I could not find any reason -- it's not used anywhere in our tree, and I
couldn't think about why one would want this, either.

Patrick