> +     ssize_t tt_query_size = sizeof(struct tt_query_packet);
> +     int i;
> +
> +     if (sizeof(struct tt_query_packet) + tt_len >
> +                                             primary_if->soft_iface->mtu) {
> +             tt_len = primary_if->soft_iface->mtu - tt_query_size;
> +             tt_len -= tt_len % sizeof(struct tt_change);
> +     }

Hi Antonio

This first struck me as being asymmetric. Then i say that
tt_query_size is sizeof(struct tt_query_packet). So use it in the if
statement, and then symmetry is restored and probably the indentation
looks better.

      Andrew

Reply via email to