summaryrefslogtreecommitdiff
path: root/Tests/AK
diff options
context:
space:
mode:
authorAndreas Kling <kling@serenityos.org>2021-11-16 01:15:21 +0100
committerAndreas Kling <kling@serenityos.org>2021-11-17 00:21:13 +0100
commit216e21a1fa95ffc87c371cdd6262467df8cdb5cd (patch)
treed58f31c31215cce39557cfcc3f35b7795334aa06 /Tests/AK
parent008355c222980bd3520caeea9848e04dc2230df1 (diff)
downloadserenity-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')
-rw-r--r--Tests/AK/TestFormat.cpp8
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");
}