> On Apr 8, 2018, at 7:53 PM, Darrell Ball <[email protected]> wrote: > > bool > parse_ipv6_ext_hdrs(const void **datap, size_t *sizep, uint8_t *nw_proto, > - uint8_t *nw_frag) > + uint8_t *nw_frag, > + const struct ovs_16aligned_ip6_frag **frag_hdr)
I know I previously acked this patch, but I do think this function could use a comment describing it, since it is a public function, and the arguments and return type aren't immediately obvious. Thanks, --Justin _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
