[ovs-dev] [PATCH v5 1/3] ofproto: change type of n_handlers and n_revalidators
Aaron Conole
aconole at redhat.com
Thu Jul 15 17:31:12 UTC 2021
Mark Gray <mark.d.gray at redhat.com> writes:
> 'n_handlers' and 'n_revalidators' are declared as type 'size_t'.
> However, dpif_handlers_set() requires parameter 'n_handlers' as
> type 'uint32_t'. This patch fixes this type mismatch.
>
> Signed-off-by: Mark Gray <mark.d.gray at redhat.com>
> Acked-by: Flavio Leitner <fbl at sysclose.org>
> ---
>
> Notes:
> v1 - Reworked based on Flavio's comments:
> * fixed inconsistency with change of size_t -> uint32_t
>
> ofproto/ofproto-dpif-upcall.c | 20 ++++++++++----------
> ofproto/ofproto-dpif-upcall.h | 5 +++--
> ofproto/ofproto-provider.h | 2 +-
> ofproto/ofproto.c | 2 +-
> 4 files changed, 15 insertions(+), 14 deletions(-)
>
I guess this is more of a cleanup - did it fix anything in practice?
Either way,
Acked-by: Aaron Conole <aconole at redhat.com>
More information about the dev
mailing list