From: Jakub Sitnicki
> Sent: 22 June 2016 12:34
...
> > -   a = attrs[TCP_METRICS_ATTR_ADDR_IPV4];
> > -   if (a) {
> > +   if ((a = attrs[TCP_METRICS_ATTR_ADDR_IPV4])) {
> 
> Copy the pointer inside the branch?
> 
> Same gain on indentation while keeping checkpatch happy.

Or as below (hacked from the mails):

        a = attrs[TCP_METRICS_ATTR_ADDR_IPV4];
        if (a) {
                if (f.daddr.family && f.daddr.family != AF_INET)
                        return 0;
                memcpy(&daddr.data, RTA_DATA(a), 4);
                daddr.bytelen = 4;
                family = AF_INET;
                atype = TCP_METRICS_ATTR_ADDR_IPV4;
                dlen = RTA_PAYLOAD(a);
        } else {
                a = attrs[TCP_METRICS_ATTR_ADDR_IPV6];
                if (!a)
                        return 0;
                memcpy(&daddr.data, RTA_DATA(a), 16);
                daddr.bytelen = 16;
                family = AF_INET6;
                atype = TCP_METRICS_ATTR_ADDR_IPV6;
                dlen = RTA_PAYLOAD(a);
        }

        David

Reply via email to