On Mon, Dec 04, 2017 at 02:37:57PM -0800, Yi-Hung Wei wrote:
> In C++, 'protected' is a keyword. This patch renames 'protected'
> to 'protected_' in a couple files so that C++ compiler will
> not get confused.
> 
> Signed-off-by: Yi-Hung Wei <[email protected]>
> ---
>  ofproto/ofproto-dpif.c | 10 +++++-----
>  ofproto/ofproto.h      |  2 +-
>  ovsdb/ovsdb-idlc.1     |  6 ++++--
>  ovsdb/ovsdb-idlc.in    | 18 ++++++++++++++++--
>  vswitchd/bridge.c      |  2 +-
>  5 files changed, 27 insertions(+), 11 deletions(-)

I don't think that any C++ code includes any of the .c or .h files that
this patch changes.  I don't think that even third-party C code should
be using these files.  If that is happening, then I want to discourage
that, not encourage it.  So, let's leave those files as-is (except as
necessary to work with the idlc changes).

C and C++ have many keywords, but this only lists a few of them.  Please
use a complete list.

Thanks,

Ben.
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to