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

AK: Make the :hex-dump format specifier print all characters

Browse source 
Previously the final line would be skipped if it was not a multiple of
|width|, this makes the character view show up for that line.
This commit is contained in:
Ali Mohammad Pur 2024-01-18 17:53:46 +03:30 committed by Andreas Kling
parent 1a07205c33
commit 4d1d88aa16
Notes: sideshowbarker 2024-07-17 06:20:50 +09:00 
Author: https://github.com/alimpfard
Commit: https://github.com/SerenityOS/serenity/commit/4d1d88aa16
Pull-request: https://github.com/SerenityOS/serenity/pull/22873
Issue: https://github.com/SerenityOS/serenity/issues/22838
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
AK/Format.cpp
View file

@ -752,7 +752,7 @@ ErrorOr<void> FormatBuilder::put_hexdump(ReadonlyBytes bytes, size_t width, char
{
auto put_char_view = [&](auto i) -> ErrorOr<void> {
TRY(put_padding(fill, 4));
for (size_t j = i - width; j < i; ++j) {
for (size_t j = i - min(i, width); j < i; ++j) {
auto ch = bytes[j];
TRY(m_builder.try_append(ch >= 32 && ch <= 127 ? ch : '.')); // silly hack
}
@ -769,7 +769,7 @@ ErrorOr<void> FormatBuilder::put_hexdump(ReadonlyBytes bytes, size_t width, char
TRY(put_u64(bytes[i], 16, false, false, true, false, Align::Right, 2));
}
if (width > 0 && bytes.size() && bytes.size() % width == 0)
if (width > 0)
TRY(put_char_view(bytes.size()));
return {};