From dc464d6efe2b644004515a66f0bdd7d28fa220f7 Mon Sep 17 00:00:00 2001 From: Carter Li Date: Tue, 28 Jul 2026 14:51:54 +0800 Subject: [PATCH] FFstrbuf: prefers `ffStrbufEnsureFree` --- src/common/FFstrbuf.h | 12 +++--------- 1 file changed, 3 insertions(+), 9 deletions(-) diff --git a/src/common/FFstrbuf.h b/src/common/FFstrbuf.h index 7b40bb42d..e3722ad82 100644 --- a/src/common/FFstrbuf.h +++ b/src/common/FFstrbuf.h @@ -221,9 +221,7 @@ static inline void ffStrbufClear(FFstrbuf* strbuf) { } static inline void ffStrbufAppendC(FFstrbuf* strbuf, char c) { - if (__builtin_expect(ffStrbufGetFree(strbuf) == 0, false)) { - ffStrbufEnsureFreeNoCheck(strbuf, 1); - } + ffStrbufEnsureFree(strbuf, 1); strbuf->chars[strbuf->length++] = c; strbuf->chars[strbuf->length] = '\0'; } @@ -232,9 +230,7 @@ static inline void ffStrbufAppendNC(FFstrbuf* strbuf, uint32_t num, char c) { if (__builtin_expect(num == 0, false)) { return; } - if (__builtin_expect(ffStrbufGetFree(strbuf) < num, false)) { - ffStrbufEnsureFreeNoCheck(strbuf, num); - } + ffStrbufEnsureFree(strbuf, num); memset(&strbuf->chars[strbuf->length], c, num); strbuf->length += num; @@ -245,9 +241,7 @@ static inline void ffStrbufAppendNS(FFstrbuf* strbuf, uint32_t length, const cha if (__builtin_expect(value == nullptr || length == 0, false)) { return; } - if (__builtin_expect(ffStrbufGetFree(strbuf) < length, false)) { - ffStrbufEnsureFreeNoCheck(strbuf, length); - } + ffStrbufEnsureFree(strbuf, length); memcpy(&strbuf->chars[strbuf->length], value, length); strbuf->length += length;