Hi Joseph,

On Sun, Nov 02, 2025 at 07:45:36PM +0100, Alejandro Colomar wrote:
> No functional change intended.
> 
> gcc/c-family/ChangeLog:
> 
>       * c-warn.cc (warn_parms_array_mismatch): Reduce scope of local
>       variable.
> 
> Signed-off-by: Alejandro Colomar <[email protected]>

Ping.  I think these 2 are the last pending patches from me.


Have a lovely night!
Alex

> ---
>  gcc/c-family/c-warn.cc | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/gcc/c-family/c-warn.cc b/gcc/c-family/c-warn.cc
> index cc127de24882..d50129f0396e 100644
> --- a/gcc/c-family/c-warn.cc
> +++ b/gcc/c-family/c-warn.cc
> @@ -3470,10 +3470,6 @@ warn_parms_array_mismatch (location_t origloc, tree 
> fndecl, tree newparms)
>       such as between f(T*) and f(T[1]), where the former mapping would be
>       empty.  */
>  
> -  /* Create an empty access specification and use it for pointers with
> -     no spec of their own.  */
> -  attr_access ptr_spec = { };
> -
>    /* Iterate over the two lists of function parameters, comparing their
>       respective mappings and diagnosing mismatches.  */
>    unsigned parmpos = 0;
> @@ -3484,6 +3480,10 @@ warn_parms_array_mismatch (location_t origloc, tree 
> fndecl, tree newparms)
>       /* Bail on invalid redeclarations with fewer arguments.  */
>       return;
>  
> +      /* Create an empty access specification and use it for pointers with
> +      no spec of their own.  */
> +      attr_access ptr_spec = { };
> +
>        /* Only check pointers and C++ references.  */
>        tree curptype = TREE_TYPE (curp);
>        tree newptype = TREE_TYPE (newp);
> -- 
> 2.51.0
> 

-- 
<https://www.alejandro-colomar.es>
Use port 80 (that is, <...:80/>).

Attachment: signature.asc
Description: PGP signature

Reply via email to