On Tue, May 21, 2024 at 02:41:27PM +0000, Karthik Nayak wrote: > Patrick Steinhardt <ps@xxxxxx> writes: > > > In `reader_seek_internal()` we either end up doing an indexed seek when > > there is one or a linear seek otherwise. These two code paths are > > Are we missing the subject here? Hm. "one" refers back to "indexed" and is supposed to mean "index" here. I agree that this is easy to misread though and may not even be proper English. It doesn't really feel worth it to rebase this series just for this issue, but if I did it would be 's/one/an index/'. Let me know in case you disagree though. Patrick
Attachment:
signature.asc
Description: PGP signature