| Message ID | 20230620222743.13900-1-cristian@rodriguez.im |
|---|---|
| State | New |
| Headers | show |
| Series | [1/5] catgets: Replace zero-length array name_ptr with C99 flexible-array. | expand |
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[]; > }; > >
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 --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[]; };