Re: [v2 PATCH 2/2] netprio_cgroup: Use memcpy instead of the for-loop to copy priomap

From: "Srivatsa S. Bhat" <redacted>
Date: Wed, 12 Sep 2012 13:54:05 +0530

On 09/12/2012 01:19 PM, David Miller wrote:

quoted

From: "Srivatsa S. Bhat" <redacted>
Date: Wed, 12 Sep 2012 11:37:47 +0530

quoted

+		memcpy(new_priomap->priomap, old_priomap->priomap,
+			old_priomap->priomap_len *
+					sizeof(old_priomap->priomap[0]));

This argument indentation is ridiculous.  Try:

		memcpy(new_priomap->priomap, old_priomap->priomap,
		       old_priomap->priomap_len *
		       sizeof(old_priomap->priomap[0]));

Using TABs exclusively for argumentat indentation is not the goal.

Rather, lining the arguments up properly so that they sit at the first
column after the first line's openning parenthesis is what you should
be trying to achieve.

OK, will fix it, thanks!

quoted

And ignoring whatever stylistic convention we may or may not have, I
find it impossibly hard to believe that the code quoted above looks
good even to you.

On second thoughts, I think the memcpy in this case will actually be worse
since it will copy the contents in chunks of smaller size than the for-loop.

I meant just coding style wise, not semantically.