[PATCH] net: ethernet: clean out braces / old code (found via checkpatch)
From: bjorn@mork.no (Bjørn Mork)
Date: 2012-10-04 18:54:29
Matthew Walster [off-list ref] writes:
On 3 October 2012 14:12, Scott Lovenberg [off-list ref] wrote:quoted
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