On 2021-02-05 Brett Okken wrote:
> After recent changes, the LZMA2OutputStream class no longer uses
> DataOutputStream, but the import statement is still present.

Fixed. Thanks!

> Now that there is a 6 byte chunkHeader, could the 1 byte tempBuf be
> removed?

It's better to keep it. It would be confusing to use the same buffer in
write(int) and writeChunk(). At glance it would look like that
writeChunk() could be overwriting the input.

-- 
Lasse Collin  |  IRC: Larhzu @ IRCnet & Freenode

Reply via email to