Re: git: 7e7f88001d7d - main - pf: use time_t for storing time_t values

From: John Baldwin <jhb_at_FreeBSD.org>
Date: Mon, 17 Feb 2025 15:24:46 UTC
On 2/14/25 12:50, Kristof Provost wrote:
> The branch main has been updated by kp:
> 
> URL: https://cgit.FreeBSD.org/src/commit/?id=7e7f88001d7dfec83cd7568369be6a587d4a51ff
> 
> commit 7e7f88001d7dfec83cd7568369be6a587d4a51ff
> Author:     Kristof Provost <kp@FreeBSD.org>
> AuthorDate: 2025-02-07 10:29:26 +0000
> Commit:     Kristof Provost <kp@FreeBSD.org>
> CommitDate: 2025-02-14 17:47:52 +0000
> 
>      pf: use time_t for storing time_t values
>      
>      No change to the underlying type, so no ABI change.
>      
>      We define __time_t as uint64_t if __LP64__, otherwise uint32_t,
>      and only define __LP64__ if long is 64 bits.
>      In other words: __time_t == long.
>      
>      ok henning@ deraadt@
>      
>      Obtained from:  OpenBSD, guenther <guenther@openbsd.org>, 6c1b69a0ff
>      Sponsored by:   Rubicon Communications, LLC ("Netgate")
>      Differential Revision:  https://reviews.freebsd.org/D48963

This is an ABI change on non-i386 32-bit platforms in FreeBSD since they
all use a 64-bit type for time_t that is not the same size as long.  Not
sure if the ABI change matters on FreeBSD though?

-- 
John Baldwin