[ovs-dev] [PATCH 07/15] datapath-windows: Replace Types.h

Nithin Raju nithin at vmware.com
Sat Aug 16 06:38:24 UTC 2014


On Aug 6, 2014, at 9:11 AM, Samuel Ghinet <sghinet at cloudbasesolutions.com> wrote:

> Replace Types.h
> 
> The coding style on windows kernel is to use uppercase for all types. This also applies to builtin types.
> We shouldn't use int64_t, uint64_t, etc. Instead, we should use windows style builtin types: INT64, UINT64, etc.
> 
> Also, the new Types.h adds macros for splitting & uniting data of variables of builtin types.
> 
> Signed-off-by: Samuel Ghinet <sghinet at cloudbasesolutions.com>


hi Sam,
The reason we defined OvsTypes.h was to make sure that we can compiled what was openvswitch.h earlier, and now is OvsDpInterface.h. There's no plan to use non windows types in the code within ovsext.

So, pls. don't delete the existing contents of OvsTypes.h. If you want to add something, pls. spin up a patch for it and send it over. I'll take a look.

thanks,
Nithin



More information about the dev mailing list