Re: [ovs-dev] [PATCH] ofp-print: #include its own header first.

2017-08-08 Thread Ben Pfaff
On Tue, Aug 08, 2017 at 04:23:10PM -0700, Joe Stringer wrote: > On 8 August 2017 at 16:11, Ben Pfaff wrote: > > The OVS coding style document says that a .c file should include the > > corresponding .h file first, to ensure that the .h file includes all of > > its dependencies, but this file didn'

Re: [ovs-dev] [PATCH] ofp-print: #include its own header first.

2017-08-08 Thread Joe Stringer
On 8 August 2017 at 16:11, Ben Pfaff wrote: > The OVS coding style document says that a .c file should include the > corresponding .h file first, to ensure that the .h file includes all of > its dependencies, but this file didn't do that. > > Signed-off-by: Ben Pfaff > --- Acked-by: Joe Stringer

[ovs-dev] [PATCH] ofp-print: #include its own header first.

2017-08-08 Thread Ben Pfaff
The OVS coding style document says that a .c file should include the corresponding .h file first, to ensure that the .h file includes all of its dependencies, but this file didn't do that. Signed-off-by: Ben Pfaff --- Depends on https://patchwork.ozlabs.org/patch/799445/. lib/ofp-print.c | 3 ++