Hi Jon,

I would welcome such a patch.

/neale

From: Jon Loeliger <j...@netgate.com>
Date: Thursday, 1 February 2018 at 17:18
To: vpp-dev <vpp-dev@lists.fd.io>, "Neale Ranns (nranns)" <nra...@cisco.com>
Subject: Consolidatation of fib_ip_proto() functions?

Neale,

I have need of using a function like fib_ip_proto(), below, in yet another file.
But rather than introduce a 5th copy of the same function, would you
accept a patch that consolidated them all into one static inline in, say,
fib_types.h where the FIB_PROTOCOL_* values are defined?

Thanks,
jdl


static inline fib_protocol_t
fib_ip_proto(bool is_ip6)
{
  return (is_ip6) ? FIB_PROTOCOL_IP6 : FIB_PROTOCOL_IP4;
}


jdl@bcc-1 $ git grep fib_ip_proto src/
src/plugins/gtpu/gtpu.c:fib_ip_proto (bool is_ip6)
src/plugins/gtpu/gtpu.c:          fib_protocol_t fp = fib_ip_proto (is_ip6);
src/plugins/gtpu/gtpu.c:          encap_fib_index = fib_table_find 
(fib_ip_proto (ipv6_set), tmp);
src/vnet/geneve/geneve.c:fib_ip_proto (bool is_ip6)
src/vnet/geneve/geneve.c:         fib_protocol_t fp = fib_ip_proto (is_ip6);
src/vnet/geneve/geneve.c:         encap_fib_index = fib_table_find 
(fib_ip_proto (ipv6_set), tmp);
src/vnet/vxlan-gpe/vxlan_gpe.c:fib_ip_proto (bool is_ip6)
src/vnet/vxlan-gpe/vxlan_gpe.c:   fib_protocol_t fp = fib_ip_proto (is_ip6);
src/vnet/vxlan/vxlan.c:fib_ip_proto(bool is_ip6)
src/vnet/vxlan/vxlan.c:          fib_protocol_t fp = fib_ip_proto(is_ip6);
src/vnet/vxlan/vxlan.c:        encap_fib_index = fib_table_find (fib_ip_proto 
(ipv6_set), tmp);

_______________________________________________
vpp-dev mailing list
vpp-dev@lists.fd.io
https://lists.fd.io/mailman/listinfo/vpp-dev

Reply via email to