Jeff King <peff@xxxxxxxx> writes:
> On Fri, Feb 07, 2025 at 12:03:36PM +0100, Patrick Steinhardt wrote:
>
>> static timestamp_t rerere_last_used_at(struct rerere_id *id)
>> {
>> + struct strbuf buf = STRBUF_INIT;
>> struct stat st;
>> + int ret;
>> +
>> + ret = stat(rerere_path(&buf, id, "postimage"), &st) ? (time_t) 0 : st.st_mtime;
>>
>> - return stat(rerere_path(id, "postimage"), &st) ? (time_t) 0 : st.st_mtime;
>> + strbuf_release(&buf);
>> + return ret;
>> }
>
> This "int ret" should surely be a timestamp_t, no?
>
> -Peff
Indeeeeeed. Thanks for a careful reading.
