diff mbox series

[1/5] catgets: Replace zero-length array name_ptr with C99 flexible-array.

Message ID 20230620222743.13900-1-cristian@rodriguez.im (mailing list archive)
State Accepted
Headers
Series [1/5] catgets: Replace zero-length array name_ptr with C99 flexible-array. |

Commit Message

Cristian Rodríguez June 20, 2023, 10:15 p.m. UTC 
  ---
 catgets/catgetsinfo.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Adhemerval Zanella Netto June 21, 2023, 12:59 p.m. UTC | #1 
LGTM, thanks.

Reviewed-by: Adhemerval Zanella  <adhemerval.zanella@linaro.org>

On 20/06/23 19:15, Cristian Rodríguez via Libc-alpha wrote:
> ---
>  catgets/catgetsinfo.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/catgets/catgetsinfo.h b/catgets/catgetsinfo.h
> index 4252e4361c..32a79a0234 100644
> --- a/catgets/catgetsinfo.h
> +++ b/catgets/catgetsinfo.h
> @@ -25,7 +25,7 @@ struct catalog_obj
>    uint32_t plane_depth;
>    /* This is in fact two arrays in one: always a pair of name and
>       pointer into the data area.  */
> -  uint32_t name_ptr[0];
> +  uint32_t name_ptr[];
>  };
>  
>
Cristian Rodríguez June 21, 2023, 3:51 p.m. UTC | #2 
I have a series of patches. Git send email is  annoying me right now.. but
I'll post first the parts that only change internal structures. Then either
grandfather-in the ones using [1] or change them in a backward compatible
way.

Cheers

El mié, 21 de jun. de 2023 08:59, Adhemerval Zanella Netto <
adhemerval.zanella@linaro.org> escribió:

> LGTM, thanks.
>
> Reviewed-by: Adhemerval Zanella  <adhemerval.zanella@linaro.org>
>
> On 20/06/23 19:15, Cristian Rodríguez via Libc-alpha wrote:
> > ---
> >  catgets/catgetsinfo.h | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/catgets/catgetsinfo.h b/catgets/catgetsinfo.h
> > index 4252e4361c..32a79a0234 100644
> > --- a/catgets/catgetsinfo.h
> > +++ b/catgets/catgetsinfo.h
> > @@ -25,7 +25,7 @@ struct catalog_obj
> >    uint32_t plane_depth;
> >    /* This is in fact two arrays in one: always a pair of name and
> >       pointer into the data area.  */
> > -  uint32_t name_ptr[0];
> > +  uint32_t name_ptr[];
> >  };
> >
> >
>
diff mbox series

Patch

diff --git a/catgets/catgetsinfo.h b/catgets/catgetsinfo.h
index 4252e4361c..32a79a0234 100644
--- a/catgets/catgetsinfo.h
+++ b/catgets/catgetsinfo.h
@@ -25,7 +25,7 @@  struct catalog_obj
   uint32_t plane_depth;
   /* This is in fact two arrays in one: always a pair of name and
      pointer into the data area.  */
-  uint32_t name_ptr[0];
+  uint32_t name_ptr[];
 };