svn commit: r320977 - head/sys/dev/ixl

Ryan Libby rlibby at gmail.com
Fri Jul 14 02:33:18 UTC 2017


On Thu, Jul 13, 2017 at 6:17 PM, Ngie Cooper <yaneurabeya at gmail.com> wrote:
> On Thu, Jul 13, 2017 at 5:10 PM, Ryan Libby <rlibby at freebsd.org> wrote:
>> Author: rlibby
>> Date: Fri Jul 14 00:10:51 2017
>> New Revision: 320977
>> URL: https://svnweb.freebsd.org/changeset/base/320977
>
> ...
>
>> +const char * const ixl_fc_string[6] = {
>> +       "None",
>> +       "Rx",
>> +       "Tx",
>> +       "Full",
>> +       "Priority",
>> +       "Default"
>> +};
>
> This should use flexible arrays, e.g.,
>
> const char * const ixl_fc_string[] = {
>     "None",
>     ...
> };
>
> -Ngie

Since this is status quo and a style point, I think I will leave this
to erj's judgement.

Ryan


More information about the svn-src-head mailing list