diff mbox series

net: core: remove unneeded semicolon

Message ID 20201101153647.2292322-1-trix@redhat.com
State New
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,