On 2022-01-31 14:12:38 +1100, Greg Nancarrow wrote: > This array was only ever meant to be read-only, and visible only to > that function. > IMO removing "static" makes things worse because now that array gets > initialized each call to the function, which is unnecessary. > I think it should just be: "static const int map_changetype_pubaction[] = ..."
Yes, static const is good. static alone, not so much.