[PATCH] net: ethernet: clean out braces / old code (found via checkpatch)

[Bjørn Mork]

Matthew Walster <matthew at walster.org> writes:
> On 3 October 2012 14:12, Scott Lovenberg <scott.lovenberg at gmail.com> wrote:
>
>> Sorry, man.  I'm not going to mix it up with Dave Miller, but really
>> if he wanted that to stay in there a comment suggesting so would have
>> been nice.
>>
>
> I have a thick skin, don't worry! I figured I was being helpful, clearly I
> caught him either at a bad time or he's just had a rash of semi-ignorant
> checkpatch-monkeys submitting and he's fed up with us!

Please note that despite the direct tone, the reply you got
 a) was prompt,
 b) explained why the patch was rejected,
 c) contained additional advice on how to proceed

You will learn to appreciate these things after a while.  The "net"
subsystem is one of the best places to send a patch, just because of the
three points above.  I am not saying that there aren't other subsystems
which are equally well maintained.  There sure are.  But the "net"
subsystem is definitely among the best, and that makes it easy to work
with for us new patch submitters.

The absolute worst thing that could have happened would have been no
answer at all, right?


Bjørn