On Thu, 22 Sep 2016 09:05:04 -0700 Stephen Hemminger <[email protected]> wrote: > On Thu, 22 Sep 2016 12:31:10 +0300 > Shmulik Ladkani <[email protected]> wrote: > > > + > > +static const char *action_name(int action) > > +{ > > + static const char * const names[] = { > > + [TCA_VLAN_ACT_POP] = "pop", > > + [TCA_VLAN_ACT_PUSH] = "push", > > + [TCA_VLAN_ACT_MODIFY] = "modify", > > + }; > > + return names[action]; > > +} > > + > > Why are you wrapping a simple array lookup in a function?
No reason in particular, was probably code evolution, will amend, thanks.
