mirror of
https://github.com/LadybirdBrowser/ladybird.git
synced 2024-09-11 05:25:32 +03:00
AK: Remove arbitrary 1 KB limit when filling a BufferedStream's buffer
When reading, we currently only fill a BufferedStream's buffer when it is empty, and only with 1 KB of data. This means that while the buffer defaults to a size of 16 KB, at least 15 KB is always unused.
This commit is contained in:
parent
8ff36e5910
commit
5c38b14045
Notes:
sideshowbarker
2024-07-16 23:17:55 +09:00
Author: https://github.com/trflynn89 Commit: https://github.com/SerenityOS/serenity/commit/5c38b14045 Pull-request: https://github.com/SerenityOS/serenity/pull/18093 Reviewed-by: https://github.com/LucasChollet
@ -231,12 +231,11 @@ private:
|
|||||||
if (m_buffer.empty_space() == 0)
|
if (m_buffer.empty_space() == 0)
|
||||||
return 0;
|
return 0;
|
||||||
|
|
||||||
// TODO: Figure out if we can do direct writes in a comfortable way.
|
|
||||||
Array<u8, 1024> temporary_buffer;
|
|
||||||
auto const fillable_slice = temporary_buffer.span().trim(min(temporary_buffer.size(), m_buffer.empty_space()));
|
|
||||||
size_t nread = 0;
|
size_t nread = 0;
|
||||||
do {
|
|
||||||
auto result = stream().read_some(fillable_slice);
|
while (true) {
|
||||||
|
auto result = m_buffer.fill_from_stream(stream());
|
||||||
|
|
||||||
if (result.is_error()) {
|
if (result.is_error()) {
|
||||||
if (!result.error().is_errno())
|
if (!result.error().is_errno())
|
||||||
return result.release_error();
|
return result.release_error();
|
||||||
@ -246,11 +245,11 @@ private:
|
|||||||
break;
|
break;
|
||||||
return result.release_error();
|
return result.release_error();
|
||||||
}
|
}
|
||||||
auto const filled_slice = result.value();
|
|
||||||
VERIFY(m_buffer.write(filled_slice) == filled_slice.size());
|
nread += result.value();
|
||||||
nread += filled_slice.size();
|
|
||||||
break;
|
break;
|
||||||
} while (true);
|
}
|
||||||
|
|
||||||
return nread;
|
return nread;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -6,6 +6,7 @@
|
|||||||
|
|
||||||
#include <AK/CircularBuffer.h>
|
#include <AK/CircularBuffer.h>
|
||||||
#include <AK/MemMem.h>
|
#include <AK/MemMem.h>
|
||||||
|
#include <AK/Stream.h>
|
||||||
|
|
||||||
namespace AK {
|
namespace AK {
|
||||||
|
|
||||||
@ -189,4 +190,20 @@ ErrorOr<void> CircularBuffer::discard(size_t discarding_size)
|
|||||||
return {};
|
return {};
|
||||||
}
|
}
|
||||||
|
|
||||||
|
ErrorOr<size_t> CircularBuffer::fill_from_stream(Stream& stream)
|
||||||
|
{
|
||||||
|
auto next_span = next_write_span();
|
||||||
|
if (next_span.size() == 0)
|
||||||
|
return 0;
|
||||||
|
|
||||||
|
auto bytes = TRY(stream.read_some(next_span));
|
||||||
|
m_used_space += bytes.size();
|
||||||
|
|
||||||
|
m_seekback_limit += bytes.size();
|
||||||
|
if (m_seekback_limit > capacity())
|
||||||
|
m_seekback_limit = capacity();
|
||||||
|
|
||||||
|
return bytes.size();
|
||||||
|
}
|
||||||
|
|
||||||
}
|
}
|
||||||
|
@ -27,6 +27,7 @@ public:
|
|||||||
size_t write(ReadonlyBytes bytes);
|
size_t write(ReadonlyBytes bytes);
|
||||||
Bytes read(Bytes bytes);
|
Bytes read(Bytes bytes);
|
||||||
ErrorOr<void> discard(size_t discarded_bytes);
|
ErrorOr<void> discard(size_t discarded_bytes);
|
||||||
|
ErrorOr<size_t> fill_from_stream(Stream&);
|
||||||
|
|
||||||
/// Compared to `read()`, this starts reading from an offset that is `distance` bytes
|
/// Compared to `read()`, this starts reading from an offset that is `distance` bytes
|
||||||
/// before the current write pointer and allows for reading already-read data.
|
/// before the current write pointer and allows for reading already-read data.
|
||||||
|
Loading…
Reference in New Issue
Block a user