From 4d2473b7fa369c4901baa6e2f3de1e64a10f0c55 Mon Sep 17 00:00:00 2001 From: Andreas Kling Date: Tue, 20 Jul 2021 13:58:38 +0200 Subject: [PATCH] Kernel: Remove confused comment in KBufferBuilder::appendff() KBufferBuilder exists for code that wants to build a KBuffer instead of a String. KBuffer is backed by anonymous VM, while String is backed by a kernel heap allocation. --- Kernel/KBufferBuilder.h | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/Kernel/KBufferBuilder.h b/Kernel/KBufferBuilder.h index 4ab6b1527d..c20c7d1548 100644 --- a/Kernel/KBufferBuilder.h +++ b/Kernel/KBufferBuilder.h @@ -31,8 +31,7 @@ public: template void appendff(CheckedFormatString&& fmtstr, const Parameters&... parameters) { - // FIXME: This is really not the way to go about it, but vformat expects a - // StringBuilder. Why does this class exist anyways? + // FIXME: This really not ideal, but vformat expects StringBuilder. StringBuilder builder; vformat(builder, fmtstr.view(), AK::VariadicFormatParams { parameters... }); append_bytes(builder.string_view().bytes());