fallintoplace opened a new pull request, #944: URL: https://github.com/apache/arrow-go/pull/944
### Rationale for this change `BitReader.fillbuffer` padded short reads with zeros without recording how many bits were present. Scalar and trailing reads could therefore treat missing input as encoded zero values. ### What changes are included in this PR? Track valid bits in each buffer, reject reads that extend beyond them, propagate seek/refill failures, validate bulk discards, and preserve zero-width reads without requiring input. ### Are these changes tested? Yes. Regression coverage includes empty input, partial scalar batches, reads crossing the 64-bit buffer boundary, and oversized discards. The suite passes with the normal implementation and with `-tags noasm`. ### Are there any user-facing changes? Truncated bit streams now return short-input errors rather than decoding missing data as zeros. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
