Re: [PATCH] perf script: Replace zero-length array with flexible-array

From: Arnaldo Melo
Date: Mon May 11 2020 - 14:49:16 EST




On May 11, 2020 2:26:20 PM GMT-03:00, "Gustavo A. R. Silva" <gustavoars@xxxxxxxxxx> wrote:
>On Mon, May 11, 2020 at 12:48:12PM -0300, Arnaldo Carvalho de Melo
>wrote:
>> Em Thu, May 07, 2020 at 02:06:15PM -0500, Gustavo A. R. Silva
>escreveu:
>> > The current codebase makes use of the zero-length array language
>> > extension to the C90 standard, but the preferred mechanism to
>declare
>> > variable-length types such as these ones is a flexible array
>member[1][2],
>> > introduced in C99:
>> >
>> > struct foo {
>> > int stuff;
>> > struct boo array[];
>> > };
>>
>> Can you split this into a kernel part and tools/ one? Also this is
>not
>> really just for "perf script", so please adjust the subject line too
>:-)
>>
>
>Sure thing. :)
>
>Is that OK if I use "perf: " for the tools part and "perf/x86: " for
>the
>kernel part?

perf tools: Summary

Since it touches multiple perf tools,

perf script: If it was only about 'perf script'
>
>Thanks
>--
>Gustavo

--
Sent from my Android device with K-9 Mail. Please excuse my brevity.