diff options
author | Andreas Kling <kling@serenityos.org> | 2021-11-16 01:15:21 +0100 |
---|---|---|
committer | Andreas Kling <kling@serenityos.org> | 2021-11-17 00:21:13 +0100 |
commit | 216e21a1fa95ffc87c371cdd6262467df8cdb5cd (patch) | |
tree | d58f31c31215cce39557cfcc3f35b7795334aa06 /Tests/AK/TestFormat.cpp | |
parent | 008355c222980bd3520caeea9848e04dc2230df1 (diff) | |
download | serenity-216e21a1fa95ffc87c371cdd6262467df8cdb5cd.zip |
AK: Convert AK::Format formatting helpers to returning ErrorOr<void>
This isn't a complete conversion to ErrorOr<void>, but a good chunk.
The end goal here is to propagate buffer allocation failures to the
caller, and allow the use of TRY() with formatting functions.
Diffstat (limited to 'Tests/AK/TestFormat.cpp')
-rw-r--r-- | Tests/AK/TestFormat.cpp | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/Tests/AK/TestFormat.cpp b/Tests/AK/TestFormat.cpp index dcae6d3c07..89324d37aa 100644 --- a/Tests/AK/TestFormat.cpp +++ b/Tests/AK/TestFormat.cpp @@ -201,9 +201,9 @@ struct B { }; template<> struct AK::Formatter<B> : Formatter<StringView> { - void format(FormatBuilder& builder, B) + ErrorOr<void> format(FormatBuilder& builder, B) { - Formatter<StringView>::format(builder, "B"); + return Formatter<StringView>::format(builder, "B"); } }; @@ -283,7 +283,7 @@ struct C { }; template<> struct AK::Formatter<C> : AK::Formatter<FormatString> { - void format(FormatBuilder& builder, C c) + ErrorOr<void> format(FormatBuilder& builder, C c) { return AK::Formatter<FormatString>::format(builder, "C(i={})", c.i); } @@ -300,7 +300,7 @@ TEST_CASE(long_long_regression) StringBuilder builder; AK::FormatBuilder fmtbuilder { builder }; - fmtbuilder.put_i64(0x0123456789abcdefLL); + MUST(fmtbuilder.put_i64(0x0123456789abcdefLL)); EXPECT_EQ(builder.string_view(), "81985529216486895"); } |