diff mbox series

net: core: remove unneeded semicolon

Message ID 20201101153647.2292322-1-trix@redhat.com (mailing list archive)
State Not Applicable
Delegated to: Netdev Maintainers
Headers show
Series net: core: remove unneeded semicolon | expand

Commit Message

Tom Rix Nov. 1, 2020, 3:36 p.m. UTC 
From: Tom Rix <trix@redhat.com>

A semicolon is not needed after a switch statement.

Signed-off-by: Tom Rix <trix@redhat.com>
---
 net/core/dev.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Andrii Nakryiko Nov. 2, 2020, 6:59 p.m. UTC | #1 
On Sun, Nov 1, 2020 at 7:38 AM <trix@redhat.com> wrote:
>
> From: Tom Rix <trix@redhat.com>
>
> A semicolon is not needed after a switch statement.
>
> Signed-off-by: Tom Rix <trix@redhat.com>
> ---

Yep.

Acked-by: Andrii Nakryiko <andrii@kernel.org>

>  net/core/dev.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/net/core/dev.c b/net/core/dev.c
> index 751e5264fd49..10f5d0c3d0d7 100644
> --- a/net/core/dev.c
> +++ b/net/core/dev.c
> @@ -8898,7 +8898,7 @@ static bpf_op_t dev_xdp_bpf_op(struct net_device *dev, enum bpf_xdp_mode mode)
>                 return dev->netdev_ops->ndo_bpf;
>         default:
>                 return NULL;
> -       };
> +       }
>  }
>
>  static struct bpf_xdp_link *dev_xdp_link(struct net_device *dev,
> --
> 2.18.1
>
Jakub Kicinski Nov. 3, 2020, 1:54 a.m. UTC | #2 
On Sun,  1 Nov 2020 07:36:47 -0800 trix@redhat.com wrote:
> From: Tom Rix <trix@redhat.com>
> 
> A semicolon is not needed after a switch statement.
> 
> Signed-off-by: Tom Rix <trix@redhat.com>

Applied...
diff mbox series

Patch

diff --git a/net/core/dev.c b/net/core/dev.c
index 751e5264fd49..10f5d0c3d0d7 100644
--- a/net/core/dev.c
+++ b/net/core/dev.c
@@ -8898,7 +8898,7 @@  static bpf_op_t dev_xdp_bpf_op(struct net_device *dev, enum bpf_xdp_mode mode)
 		return dev->netdev_ops->ndo_bpf;
 	default:
 		return NULL;
-	};
+	}
 }
 
 static struct bpf_xdp_link *dev_xdp_link(struct net_device *dev,