On Tue, Apr 6, 2021 at 1:11 PM Conor Walsh <conor.wa...@intel.com> wrote:
> +        /* populate the FIB table */
> +
> +        for (i = 0; i < IPV4_L3FWD_NUM_ROUTES; i++) {
> +            struct in_addr in;
> +
> +            /* skip unused ports */
> +            if ((1 << ipv4_l3fwd_fib_route_array[i].if_out & 
> enabled_port_mask) == 0)


Little extra comment, this code is not using the same variables as the code.

Pasting all this code in the documentation does not help, I would just
remove it and/or reference the code example to avoid doc and example
being out of sync in the future.


> +                continue;
> +


-- 
David Marchand

Reply via email to