Re: [PATCH 2/5] streaming.c: remove enum/function/vtbl indirection

Jeff King [off-list ref] writes:

quoted

We can thus remove the entire enum, and the "open_istream_tbl" virtual
table that (indirectly) referenced it.

Yeah, I think this is simpler. The value of the vtable was that we might
have added more functions to it, but we haven't done so over the course
of the last 10 years. And I have trouble imagining for what purpose we
would. So it seems like unnecessary complexity.

The code hasn't changed all that much since it was written, so
another equally plausible way forward is not to touch it at all ;-),
but I am fine with this simplification, too.