çå: çå: [PATCH 1/1] netfilter: Add helper array register/unregister functions
Date: Wed Jul 20 2016 - 05:11:16 EST
Ok, I get it.
Another question, Liping found the port style of sip, irc, and tftp was different with ftp.
It should be use iterator "i" to generate name for them.
Then should I update the original patch or create another one to correct it?
BTW, I am sorry about that. I didn't notice there are two port styles.
åää: Pablo Neira Ayuso [mailto:pablo@xxxxxxxxxxxxx]
åéæé: 2016å7æ20æ 16:50
æää: éå <fgao@xxxxxxxxxx>
æé: kaber@xxxxxxxxx; netfilter-devel@xxxxxxxxxxxxxxx; netdev@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; gfree.wind@xxxxxxxxx
äé: Re: çå: [PATCH 1/1] netfilter: Add helper array register/unregister functions
On Wed, Jul 20, 2016 at 08:31:13AM +0800, éå wrote:
> Thanks Pablo.
> I had used the script "checkpatch.pl" to check the patch file.
> There was no indentation error reported.
> So could you give me more tails please or point one indentation error?
> Then I could correct it by myself next time.
I'm refering to this specifically:
static int function(int parameter1, struct another_structure *blah,
int parameter2, unsigned int parameter3);
It is just a comestic issue, but we consistently align function parameters to the initial parens.
As I said, I have just manually fixed this here, so no problem, just keep this in mind for the next time.
Another observation: You should bump patch version numbering in each revision and keep some history on its evolution.
The area after the patch separator --- and before diff stats is good to place volatile information that is only meaningful to the review process, I mean something like this:
subsystem: Patch title.
Signed-off-by: Lucas Skywalker <trotacielos@xxxxxxxxxxxxx>
v3: Address comments from Chebakia on possible backward compatibility
v2: New parameter to control something.
v1: Initial patch.
include/net/netfilter/nf_tables.h | 25 ++-
diff --git a/include/net/netfilter/nf_tables.h b/include/net/netfilter/nf_tables.h