Thread (11 messages) 11 messages, 4 authors, 2012-10-04
DORMANTno replies

[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
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help