Re: [ovs-dev] [v2] ofproto: Use macros to define DPIF support fields

2017-04-03 Thread Andy Zhou
On Mon, Apr 3, 2017 at 5:30 PM, Joe Stringer wrote: > On 3 April 2017 at 13:30, Andy Zhou wrote: >> When adding a new field in the 'struct dpif_backer_support', the >> corresponding appctl show command should be updated to display >> the new field. Currently, there

Re: [ovs-dev] [v2] ofproto: Use macros to define DPIF support fields

2017-04-03 Thread Joe Stringer
On 3 April 2017 at 13:30, Andy Zhou wrote: > When adding a new field in the 'struct dpif_backer_support', the > corresponding appctl show command should be updated to display > the new field. Currently, there is nothing to remind the developer > that to update the show command.

[ovs-dev] [v2] ofproto: Use macros to define DPIF support fields

2017-04-03 Thread Andy Zhou
When adding a new field in the 'struct dpif_backer_support', the corresponding appctl show command should be updated to display the new field. Currently, there is nothing to remind the developer that to update the show command. This can lead to code maintenance issues. Switch to use macros to