RGBCube/serenity
1
Fork 0
mirror of https://github.com/RGBCube/serenity synced 2025-06-02 17:38:12 +00:00
Code Issues Activity Actions

LibCompress: Remove special casing for looping DEFLATE seekbacks

Browse source 
The `copy_from_seekback` method already handles this exactly as DEFLATE
expects, but it is slightly more optimized.
This commit is contained in:
Tim Schumacher 2023-04-20 21:41:14 +02:00 • committed by Jelle Raaijmakers
parent 00ac73be57
commit dffef6bb71
1 changed files with 3 additions and 11 deletions
Download patch file Download diff file Expand all files Collapse all files

14
Userland/Libraries/LibCompress/Deflate.cpp
View file

@ -211,18 +211,10 @@ ErrorOr<bool> DeflateDecompressor::CompressedBlock::try_read_more()
auto const distance = TRY(m_decompressor.decode_distance(distance_symbol)); auto const distance = TRY(m_decompressor.decode_distance(distance_symbol));
if (distance < length) { auto copied_length = TRY(m_decompressor.m_output_buffer.copy_from_seekback(distance, length));
for (size_t idx = 0; idx < length; ++idx) {
u8 byte = 0;
TRY(m_decompressor.m_output_buffer.read_with_seekback({ &byte, sizeof(byte) }, distance));
m_decompressor.m_output_buffer.write({ &byte, sizeof(byte) });
}
} else {
auto copied_length = TRY(m_decompressor.m_output_buffer.copy_from_seekback(distance, length));
// TODO: What should we do if the output buffer is full? // TODO: What should we do if the output buffer is full?
VERIFY(copied_length == length); VERIFY(copied_length == length);
}
return true; return true;
} }