[PATCH nf-next 4/6] netfilter-ipv4: function definition layout

2015-10-14 Thread Ian Morris
Use tabs instead of spaces to indent second line of parameters in function definitions. No changes detected by objdiff. Signed-off-by: Ian Morris <i...@chirality.org.uk> --- net/ipv4/netfilter/arp_tables.c | 6 +++--- net/ipv4/netfilter/ip_tables.c | 6 +++--- 2 files changed, 6 inse

[PATCH nf-next 6/6] netfilter-ipv4: whitespace around operators

2015-10-14 Thread Ian Morris
This patch cleanses whitespace around arithmetical operators. No changes detected by objdiff. Signed-off-by: Ian Morris <i...@chirality.org.uk> --- net/ipv4/netfilter/ipt_CLUSTERIP.c | 8 net/ipv4/netfilter/ipt_ah.c| 2 +- net/ipv4/netfilter/nf_nat_snmp_basic

[PATCH nf-next 2/6] netfilter-ipv4: label placement

2015-10-14 Thread Ian Morris
Whitespace cleansing: Labels should not be indented. No changes detected by objdiff. Signed-off-by: Ian Morris <i...@chirality.org.uk> --- net/ipv4/netfilter/arp_tables.c | 2 +- net/ipv4/netfilter/ip_tables.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/ne

[PATCH nf-next 1/6] netfilter-ipv4: Line layout whitespace fixes

2015-10-14 Thread Ian Morris
Cleanses some whitespace issues by removing a leading space before a tab. No changes detected by objdiff. Signed-off-by: Ian Morris <i...@chirality.org.uk> --- net/ipv4/netfilter/ipt_ECN.c | 2 +- net/ipv4/netfilter/nf_nat_pptp.c | 2 +- net/ipv4/netfilter/nf_nat_snmp_b

[PATCH nf-next 3/6] netfilter-ipv4: ternary operator layout

2015-10-14 Thread Ian Morris
Correct whitespace layout of ternary operators in the netfilter-ipv4 code. No changes detected by objdiff. Signed-off-by: Ian Morris <i...@chirality.org.uk> --- net/ipv4/netfilter/arp_tables.c | 4 ++-- net/ipv4/netfilter/ip_tables.c | 6 +++--- 2 files changed, 5 insertions(+), 5 del

[PATCH nf-next 0/6] coding style improvements: netfilter-ipv4

2015-10-14 Thread Ian Morris
This series of patches improves the coding style of the netfilter-ipv4 code by addressing some issues detected by checkpatch. The changes were previously submitted as part of a larger monolithic patch but on advice from Pablo, these are being re-sent in smaller, more structured batches. Ian

[PATCH nf-next 5/6] netfilter-ipv4: code indentation

2015-10-14 Thread Ian Morris
Use tabs instead of spaces to indent code. No changes detected by objdiff. Signed-off-by: Ian Morris <i...@chirality.org.uk> --- net/ipv4/netfilter/ip_tables.c| 6 +++--- net/ipv4/netfilter/ipt_SYNPROXY.c | 2 +- net/ipv4/netfilter/iptable_security.c | 2 +- 3 files chan

[PATCH nf-next 7/7] netfilter-ipv6: pointer cast layout

2015-10-11 Thread Ian Morris
Correct whitespace layout of a pointer casting. No changes detected by objdiff. Signed-off-by: Ian Morris <i...@chirality.org.uk> --- net/ipv6/netfilter/nf_conntrack_reasm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/net/ipv6/netfilter/nf_conntrack_reasm.c b/ne

[PATCH nf-next 5/7] netfilter-ipv6: ternary operator layout

2015-10-11 Thread Ian Morris
Correct whitespace layout of ternary operators in the netfilter-ipv6 code. No changes detected by objdiff. Signed-off-by: Ian Morris <i...@chirality.org.uk> --- net/ipv6/netfilter/ip6_tables.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/net/ipv6/net

[PATCH nf-next 2/7] netfilter-ipv6: function definition layout

2015-10-11 Thread Ian Morris
Use tabs instead of spaces to indent second line of parameters in function definitions. No changes detected by objdiff. Signed-off-by: Ian Morris <i...@chirality.org.uk> --- net/ipv6/netfilter/ip6_tables.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/ne

[PATCH nf-next 4/7] netfilter-ipv6: whitespace around operators

2015-10-11 Thread Ian Morris
This patch cleanses whitespace around arithmetical operators. No changes detected by objdiff. Signed-off-by: Ian Morris <i...@chirality.org.uk> --- net/ipv6/netfilter/nf_conntrack_proto_icmpv6.c | 4 ++-- net/ipv6/netfilter/nf_conntrack_reasm.c| 2 +- net/ipv6/netfilter/nf_reject_

[PATCH nf-next 6/7] netfilter-ipv6: improve if statements

2015-10-11 Thread Ian Morris
Correct whitespace layout of if statements. No changes detected by objdiff. Signed-off-by: Ian Morris <i...@chirality.org.uk> --- net/ipv6/netfilter/ip6_tables.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/net/ipv6/netfilter/ip6_tables.c b/net/ipv6/net

[PATCH nf-next 1/7] netfilter-ipv6: label placement

2015-10-11 Thread Ian Morris
Whitespace cleansing: Labels should not be indented. No changes detected by objdiff. Signed-off-by: Ian Morris <i...@chirality.org.uk> --- net/ipv6/netfilter/ip6_tables.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/net/ipv6/netfilter/ip6_tables.c b/net/ipv6/net

[PATCH nf-next 3/7] netfilter-ipv6: code indentation

2015-10-11 Thread Ian Morris
Use tabs instead of spaces to indent code. No changes detected by objdiff. Signed-off-by: Ian Morris <i...@chirality.org.uk> --- net/ipv6/netfilter/ip6_tables.c | 4 ++-- net/ipv6/netfilter/ip6t_SYNPROXY.c| 2 +- net/ipv6/netfilter/nft_chain_route_ipv6.c | 6 +++--- 3

[PATCH nf-next 0/7] coding style improvements for netfilter

2015-10-11 Thread Ian Morris
This series of patches improves the coding style of the netfilter-ipv6 code by addressing some issues detected by checkpatch. The changes were previously submitted as part of a larger monolithic patch but on advice from Pablo, these are being re-sent in smaller, more structured batches. Ian

Re: [PATCH nf-next] netfilter: Line layout whitespace fixes

2015-10-01 Thread Ian Morris
On 30/09/15 23:17, Pablo Neira Ayuso wrote: On Thu, Sep 24, 2015 at 11:10:06AM +0100, Ian Morris wrote: Fixes various whitespace issues detected by checkpatch (e.g. adding spaces between operations, replacing spaces with tabs, etc) No changes detected by objdiff. These large cleanup patch

[PATCH nf-next] netfilter: Line layout whitespace fixes

2015-09-24 Thread Ian Morris
Fixes various whitespace issues detected by checkpatch (e.g. adding spaces between operations, replacing spaces with tabs, etc) No changes detected by objdiff. Signed-off-by: Ian Morris <i...@chirality.org.uk> --- net/bridge/netfilter/ebt_among.c | 2 +- net/ipv4/net

[PATCH net-next] ipv6: trivial whitespace fix

2015-08-14 Thread Ian Morris
Change brace placement to be in line with coding standards Signed-off-by: Ian Morris i...@chirality.org.uk --- net/ipv6/udp.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/net/ipv6/udp.c b/net/ipv6/udp.c index e51fc3e..0aba654 100644 --- a/net/ipv6/udp.c +++ b/net/ipv6

[PATCH net-next] ipv6: remove unnecessary include

2015-08-14 Thread Ian Morris
printk.h does not need to be explicitly included as we include kernel.h which already called it. Signed-off-by: Ian Morris i...@chirality.org.uk --- net/ipv6/ip6_offload.c | 1 - 1 file changed, 1 deletion(-) diff --git a/net/ipv6/ip6_offload.c b/net/ipv6/ip6_offload.c index 08b6204..1cb2dc7

[PATCH net-next 3/6] ipv6: netfilter: remove unnecessary braces

2015-04-25 Thread Ian Morris
Remove braces from a few if statements where not required. No changes detected by objdiff. Signed-off-by: Ian Morris i...@chirality.org.uk --- net/ipv6/netfilter/ip6_tables.c | 3 +-- net/ipv6/netfilter/nf_conntrack_reasm.c | 4 ++-- 2 files changed, 3 insertions(+), 4 deletions

[PATCH net-next 4/6] ipv6: netfilter: comparison for equality with NULL

2015-04-25 Thread Ian Morris
The code uses a mixture of coding styles. In some instances check for NULL pointer is done as x == NULL and sometimes as !x. !x is preferred according to checkpatch and this patch makes the code consistent by adopting the latter form. No changes detected by objdiff. Signed-off-by: Ian Morris i

[PATCH net-next 6/6] ipv6: netfilter: simply if-else statements

2015-04-25 Thread Ian Morris
Remove else branch of if statements where a return is always done when the condition is true. No changes detected by objdiff. Signed-off-by: Ian Morris i...@chirality.org.uk --- net/ipv6/netfilter/ip6_tables.c| 7 +- net/ipv6/netfilter/ip6t_hbh.c | 105

[PATCH net-next 5/6] ipv6: netfilter: comparison for inequality with NULL

2015-04-25 Thread Ian Morris
The code uses a mixture of coding styles. In some instances check for NULL pointer is done as x != NULL and sometimes as x. x is preferred according to checkpatch and this patch makes the code consistent by adopting the latter form. No changes detected by objdiff. Signed-off-by: Ian Morris i

[PATCH net-next 2/6] ipv6: netfilter: whitespace in line layouts

2015-04-25 Thread Ian Morris
Try to improve coding readability by adding spaces in line layouts. No changes to functionality. No changes according to objdiff. Signed-off-by: Ian Morris i...@chirality.org.uk --- net/ipv6/netfilter/ip6_tables.c| 12 ++-- net/ipv6/netfilter/ip6t_rpfilter.c

[PATCH net-next 0/6] ipv6: netfilter - coding style improvements

2015-04-25 Thread Ian Morris
This series of patches removes some style issues in the ipv6 netfilter code detected by checkpatch. There is no change to functionality and no changes to the resultant object determined by objdiff. Ian Morris (6): ipv6: netfilter: Tidy up indenting ipv6: netfilter: whitespace in line layouts

[PATCH net-next 1/6] ipv6: netfilter: Tidy up indenting

2015-04-25 Thread Ian Morris
This patch cleanses some indent issues in the ipv6 netfilter code by converting spaces to tabs etc. No changes to functionality. No changes according to objdiff. Signed-off-by: Ian Morris i...@chirality.org.uk --- net/ipv6/netfilter/ip6_tables.c | 12 ++-- net/ipv6/netfilter