diff options
author | Andreas Kling <kling@serenityos.org> | 2021-11-10 14:33:44 +0100 |
---|---|---|
committer | Andreas Kling <kling@serenityos.org> | 2021-11-10 21:58:58 +0100 |
commit | a15ed8743d03c6c683f19447be20ca7dac768485 (patch) | |
tree | fe3b808b4909686757dae5c4a949ba18fe7e5eba /Userland/Libraries/LibPDF/Parser.cpp | |
parent | 88b6428c25ea046a4bb19bb6f3f68dd4f1439539 (diff) | |
download | serenity-a15ed8743d03c6c683f19447be20ca7dac768485.zip |
AK: Make ByteBuffer::try_* functions return ErrorOr<void>
Same as Vector, ByteBuffer now also signals allocation failure by
returning an ENOMEM Error instead of a bool, allowing us to use the
TRY() and MUST() patterns.
Diffstat (limited to 'Userland/Libraries/LibPDF/Parser.cpp')
-rw-r--r-- | Userland/Libraries/LibPDF/Parser.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/Userland/Libraries/LibPDF/Parser.cpp b/Userland/Libraries/LibPDF/Parser.cpp index 1872c0e9b8..a591060c47 100644 --- a/Userland/Libraries/LibPDF/Parser.cpp +++ b/Userland/Libraries/LibPDF/Parser.cpp @@ -273,8 +273,8 @@ bool Parser::initialize_hint_tables() if (!buffer_result.has_value()) return false; possible_merged_stream_buffer = buffer_result.release_value(); - auto ok = possible_merged_stream_buffer.try_append(primary_hint_stream->bytes()); - ok = ok && possible_merged_stream_buffer.try_append(overflow_hint_stream->bytes()); + auto ok = !possible_merged_stream_buffer.try_append(primary_hint_stream->bytes()).is_error(); + ok = ok && !possible_merged_stream_buffer.try_append(overflow_hint_stream->bytes()).is_error(); if (!ok) return false; hint_stream_bytes = possible_merged_stream_buffer.bytes(); |