linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] net: Remove some unneeded semicolon
@ 2018-08-04 11:41 zhong jiang
  2018-08-04 18:08 ` Sergei Shtylyov
  2018-08-04 20:05 ` David Miller
  0 siblings, 2 replies; 4+ messages in thread
From: zhong jiang @ 2018-08-04 11:41 UTC (permalink / raw)
  To: davem, wensong, horms, ja; +Cc: netdev, linux-kernel

That semicolon is not unneeded , Just remove them.

Signed-off-by: zhong jiang <zhongjiang@huawei.com>
---
 net/core/utils.c               | 2 +-
 net/netfilter/ipvs/ip_vs_ctl.c | 2 +-
 net/packet/af_packet.c         | 4 ++--
 net/sunrpc/auth_gss/auth_gss.c | 2 +-
 4 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/net/core/utils.c b/net/core/utils.c
index d47863b..2a597ac 100644
--- a/net/core/utils.c
+++ b/net/core/utils.c
@@ -397,7 +397,7 @@ int inet_pton_with_scope(struct net *net, __kernel_sa_family_t af,
 		break;
 	default:
 		pr_err("unexpected address family %d\n", af);
-	};
+	}
 
 	return ret;
 }
diff --git a/net/netfilter/ipvs/ip_vs_ctl.c b/net/netfilter/ipvs/ip_vs_ctl.c
index dd21782..62eefea 100644
--- a/net/netfilter/ipvs/ip_vs_ctl.c
+++ b/net/netfilter/ipvs/ip_vs_ctl.c
@@ -134,7 +134,7 @@ static void update_defense_level(struct netns_ipvs *ipvs)
 		} else {
 			atomic_set(&ipvs->dropentry, 0);
 			ipvs->sysctl_drop_entry = 1;
-		};
+		}
 		break;
 	case 3:
 		atomic_set(&ipvs->dropentry, 1);
diff --git a/net/packet/af_packet.c b/net/packet/af_packet.c
index e3e00d3..345e380 100644
--- a/net/packet/af_packet.c
+++ b/net/packet/af_packet.c
@@ -1582,7 +1582,7 @@ static int fanout_set_data(struct packet_sock *po, char __user *data,
 		return fanout_set_data_ebpf(po, data, len);
 	default:
 		return -EINVAL;
-	};
+	}
 }
 
 static void fanout_release_data(struct packet_fanout *f)
@@ -1591,7 +1591,7 @@ static void fanout_release_data(struct packet_fanout *f)
 	case PACKET_FANOUT_CBPF:
 	case PACKET_FANOUT_EBPF:
 		__fanout_set_data_bpf(f, NULL);
-	};
+	}
 }
 
 static bool __fanout_id_is_free(struct sock *sk, u16 candidate_id)
diff --git a/net/sunrpc/auth_gss/auth_gss.c b/net/sunrpc/auth_gss/auth_gss.c
index 3e192e2..846ea65 100644
--- a/net/sunrpc/auth_gss/auth_gss.c
+++ b/net/sunrpc/auth_gss/auth_gss.c
@@ -517,7 +517,7 @@ static int gss_encode_v1_msg(struct gss_upcall_msg *gss_msg,
 		err = gss_encode_v1_msg(gss_msg, service_name, gss_auth->target_name);
 		if (err)
 			goto err_put_pipe_version;
-	};
+	}
 	kref_get(&gss_auth->kref);
 	return gss_msg;
 err_put_pipe_version:
-- 
1.7.12.4


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] net: Remove some unneeded semicolon
  2018-08-04 11:41 [PATCH] net: Remove some unneeded semicolon zhong jiang
@ 2018-08-04 18:08 ` Sergei Shtylyov
  2018-08-05 13:29   ` zhong jiang
  2018-08-04 20:05 ` David Miller
  1 sibling, 1 reply; 4+ messages in thread
From: Sergei Shtylyov @ 2018-08-04 18:08 UTC (permalink / raw)
  To: zhong jiang, davem, wensong, horms, ja; +Cc: netdev, linux-kernel

On 08/04/2018 02:41 PM, zhong jiang wrote:

> That semicolon is not unneeded , Just remove them.

   s/un//, right? ;-)
   That and you suddenly switch from singular to plural.

> Signed-off-by: zhong jiang <zhongjiang@huawei.com>

MBR, Sergei

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] net: Remove some unneeded semicolon
  2018-08-04 11:41 [PATCH] net: Remove some unneeded semicolon zhong jiang
  2018-08-04 18:08 ` Sergei Shtylyov
@ 2018-08-04 20:05 ` David Miller
  1 sibling, 0 replies; 4+ messages in thread
From: David Miller @ 2018-08-04 20:05 UTC (permalink / raw)
  To: zhongjiang; +Cc: wensong, horms, ja, netdev, linux-kernel

From: zhong jiang <zhongjiang@huawei.com>
Date: Sat, 4 Aug 2018 19:41:41 +0800

> That semicolon is not unneeded , Just remove them.
> 
> Signed-off-by: zhong jiang <zhongjiang@huawei.com>

Applied to net-next, thanks.

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] net: Remove some unneeded semicolon
  2018-08-04 18:08 ` Sergei Shtylyov
@ 2018-08-05 13:29   ` zhong jiang
  0 siblings, 0 replies; 4+ messages in thread
From: zhong jiang @ 2018-08-05 13:29 UTC (permalink / raw)
  To: Sergei Shtylyov; +Cc: davem, wensong, horms, ja, netdev, linux-kernel

On 2018/8/5 2:08, Sergei Shtylyov wrote:
> On 08/04/2018 02:41 PM, zhong jiang wrote:
>
>> That semicolon is not unneeded , Just remove them.
>    s/un//, right? ;-)
>    That and you suddenly switch from singular to plural.
 :-[ I guess that David has corrected this.

 Thanks
 zhong jiang
>> Signed-off-by: zhong jiang <zhongjiang@huawei.com>
> MBR, Sergei
>
>



^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2018-08-05 13:30 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-08-04 11:41 [PATCH] net: Remove some unneeded semicolon zhong jiang
2018-08-04 18:08 ` Sergei Shtylyov
2018-08-05 13:29   ` zhong jiang
2018-08-04 20:05 ` David Miller

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).