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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Matthew Walster <matthew@xxxxxxxxxxx> writes:
> On 3 October 2012 14:12, Scott Lovenberg <scott.lovenberg@xxxxxxxxx> 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

_______________________________________________
Kernelnewbies mailing list
Kernelnewbies@xxxxxxxxxxxxxxxxx
http://lists.kernelnewbies.org/mailman/listinfo/kernelnewbies



[Index of Archives]     [Newbies FAQ]     [Linux Kernel Mentors]     [Linux Kernel Development]     [IETF Annouce]     [Git]     [Networking]     [Security]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux SCSI]     [Linux ACPI]
  Powered by Linux