beenull/ladybird
1
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2024-11-26 09:30:24 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 7

LibGfx/JPEGWriter: Make vertical_macroblocks a local

Browse source 
It's only used in one function.

No behavior change.
This commit is contained in:
Nico Weber 2024-01-31 20:53:19 -05:00 committed by Andreas Kling
parent 8964a52fe0
commit ad7d25f089
Notes: sideshowbarker 2024-07-17 06:38:11 +09:00 
Author: https://github.com/nico
Commit: https://github.com/SerenityOS/serenity/commit/ad7d25f089
Pull-request: https://github.com/SerenityOS/serenity/pull/23034
Reviewed-by: https://github.com/LucasChollet ✅
1 changed files with 2 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
Userland/Libraries/LibGfx/ImageFormats/JPEGWriter.cpp
View file

@ -102,9 +102,8 @@ public:
ErrorOr<void> initialize_mcu(Bitmap const& bitmap)
{
u64 const horizontal_macroblocks = ceil_div(bitmap.width(), 8);
m_vertical_macroblocks = ceil_div(bitmap.height(), 8);
TRY(m_macroblocks.try_resize(horizontal_macroblocks * m_vertical_macroblocks));
u64 const vertical_macroblocks = ceil_div(bitmap.height(), 8);
TRY(m_macroblocks.try_resize(horizontal_macroblocks * vertical_macroblocks));
for (u16 y {}; y < bitmap.height(); ++y) {
u16 const vertical_macroblock_index = y / 8;
@ -339,8 +338,6 @@ private:
Vector<Macroblock> m_macroblocks {};
Array<i16, 3> m_last_dc_values {};
u64 m_vertical_macroblocks {};
JPEGBigEndianOutputBitStream m_bit_stream;
};