mirrors/ladybird
0
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2025-09-01 23:26:36 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 10

LibGfx/GIF: Make write_logical_descriptor() take an IntSize

Browse source 
The bitmap was only needed for its size.
This commit is contained in:
Lucas CHOLLET 2024-05-18 20:24:39 -04:00 • committed by Andreas Kling
commit 6a5f8b5163
Notes: sideshowbarker 2024-07-17 07:20:49 +09:00
1 changed files with 5 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

10
Userland/Libraries/LibGfx/ImageFormats/GIFWriter.cpp
View file

@ -21,15 +21,15 @@ ErrorOr<void> write_header(Stream& stream)
return {};
}
ErrorOr<void> write_logical_descriptor(BigEndianOutputBitStream& stream, Bitmap const& bitmap)
ErrorOr<void> write_logical_descriptor(BigEndianOutputBitStream& stream, IntSize size)
{
// 18. Logical Screen Descriptor
if (bitmap.width() > NumericLimits<u16>::max() || bitmap.height() > NumericLimits<u16>::max())
if (size.width() > NumericLimits<u16>::max() || size.height() > NumericLimits<u16>::max())
return Error::from_string_literal("Bitmap size is too big for a GIF");
TRY(stream.write_value<u16>(bitmap.width()));
TRY(stream.write_value<u16>(bitmap.height()));
TRY(stream.write_value<u16>(size.width()));
TRY(stream.write_value<u16>(size.height()));
// Global Color Table Flag
TRY(stream.write_bits(false, 1));
@ -135,7 +135,7 @@ ErrorOr<void> GIFWriter::encode(Stream& stream, Bitmap const& bitmap)
TRY(write_header(stream));
BigEndianOutputBitStream bit_stream { MaybeOwned<Stream> { stream } };
TRY(write_logical_descriptor(bit_stream, bitmap));
TRY(write_logical_descriptor(bit_stream, bitmap.size()));
// Write a Table-Based Image
TRY(write_image_descriptor(bit_stream, bitmap));