On Fri, Nov 03, 2017 at 03:45:07AM CDT, Paul Eggert wrote:
Zev Weiss wrote:

-static struct
+static const struct
 {
   char const name[12];

There's already a const after the } at the end of the struct, so this 'const' is not needed. The usual style in 'grep' is to put the 'const' after the type, not before it.

Come to think of it, the 'const' in front of 'name' is also a duplicate, so I installed the attached.

Ah, missed that, sorry (had seen it written as 'static const struct'
earlier in the file on 'color dict') -- nevermind on this one, obviously.

Zev




Reply via email to