> On 20 Aug 2026, at 14:53, Aleksander Alekseev <[email protected]> > wrote: > > Hi Daniel, > >> I'm not sure that using TAR_BLOCK_SIZE is preferred for >> a lookahead buffer used to identify the archive format where tar is just one >> of >> the options. >> >> I would be more inclined to initialize .lookaheadSize like today and then use >> that instead of the hardcoded 512. Having a buffer in ArchiveHandle without >> knowing the size of it does seem like a bad practice even if nothing uses it >> today. > > Sounds good. Here is the patch v2.
Seems reasonable. Are there any consumers of the buffer who hardcode an assumption of its size which could use .lookaheadSize instead? I'll revist this on Sunday when I get back in the office unless beaten to it. -- Daniel Gustafsson
