Re: [PATCH v2] REFTABLE_REALLOC_ARRAY: remove this unsafe yet unused macro

From: lidongyan <502024330056@smail.nju.edu.cn>
Date: May 9, 2025, 07:38 UTC
Message-ID: <02E0C9C6-3A1A-480C-834A-BB30C3170C78@smail.nju.edu.cn>
In-Reply-To: <aB2l_WaBFyz5POld@pks.im>

2025年5月9日 14:51，Patrick Steinhardt <ps@pks.im> 写道：

Show 30 quoted lines

> 
> On Fri, May 09, 2025 at 02:04:22AM +0000, Lidong Yan via GitGitGadget wrote:
>> diff --git a/reftable/basics.h b/reftable/basics.h
>> index d8888c12629..667feffd935 100644
>> --- a/reftable/basics.h
>> +++ b/reftable/basics.h
>> @@ -199,16 +199,8 @@ static inline int reftable_alloc_size(size_t nelem, size_t elsize, size_t *out)
>> (x) = reftable_malloc(alloc_size); \
>> } \
>> } while (0)
>> -#define REFTABLE_CALLOC_ARRAY(x, alloc) (x) = reftable_calloc((alloc), sizeof(*(x)))
>> -#define REFTABLE_REALLOC_ARRAY(x, alloc) do { \
>> - size_t alloc_size; \
>> - if (reftable_alloc_size(sizeof(*(x)), (alloc), &alloc_size) < 0) { \
>> - errno = ENOMEM; \
>> - (x) = NULL; \
>> - } else { \
>> - (x) = reftable_realloc((x), alloc_size); \
>> - } \
>> - } while (0)
>> +#define REFTABLE_CALLOC_ARRAY(x, alloc) \
>> + (x) = reftable_calloc((alloc), sizeof(*(x)))
> 
> Let's avoid reformatting unrelated macros. But other than that I fully
> agree -- we should remove stuff that we don't use in the first place.
> 
> Thanks!
> 
> Patrick
>

Ok, I will restore REFTABLE_CALLOC_ARRAY in the next patch.