Re: git: 0f2090ccfeb6 - main - kassert: Explicitly include <sys/_types.h>

From: Konstantin Belousov <kostikbel_at_gmail.com>
Date: Thu, 08 May 2025 00:35:41 UTC
On Mon, Apr 28, 2025 at 12:23:42PM +0000, Olivier Certner wrote:
> The branch main has been updated by olce:
> 
> URL: https://cgit.FreeBSD.org/src/commit/?id=0f2090ccfeb6e3e1a2290300b53baedfb057c2b5
> 
> commit 0f2090ccfeb6e3e1a2290300b53baedfb057c2b5
> Author:     Olivier Certner <olce@FreeBSD.org>
> AuthorDate: 2025-04-07 12:16:34 +0000
> Commit:     Olivier Certner <olce@FreeBSD.org>
> CommitDate: 2025-04-28 12:19:42 +0000
> 
>     kassert: Explicitly include <sys/_types.h>
>     
>     Include it as <sys/kassert.h> has direct references defined in it (to
>     '__va_list' and '__printflike' at least).
>     
>     This is a step to make <sys/kassert.h> usable without the need to
>     explicitly include other headers.
>     
>     Reviewed by:    imp, markj
>     MFC after:      3 days
>     Sponsored by:   The FreeBSD Foundation
>     Differential Revision:  https://reviews.freebsd.org/D49972
> ---
>  sys/sys/kassert.h | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/sys/sys/kassert.h b/sys/sys/kassert.h
> index a2bc42058d93..e61eca308c07 100644
> --- a/sys/sys/kassert.h
> +++ b/sys/sys/kassert.h
> @@ -31,6 +31,7 @@
>  #ifndef _SYS_KASSERT_H_
>  #define	_SYS_KASSERT_H_
>  
> +#include <sys/_types.h>
>  #include <sys/cdefs.h>
cdefs.h must go first, I believe.

>  
>  #ifdef _KERNEL