2024-03-15 09:27:09

by Colin Ian King

[permalink] [raw]
Subject: [PATCH][next] selftests/bpf: Remove second semicolon

There are statements with two semicolons. Remove the second one, it
is redundant.

Signed-off-by: Colin Ian King <[email protected]>
---
tools/testing/selftests/bpf/benchs/bench_local_storage_create.c | 2 +-
tools/testing/selftests/bpf/progs/iters.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/tools/testing/selftests/bpf/benchs/bench_local_storage_create.c b/tools/testing/selftests/bpf/benchs/bench_local_storage_create.c
index b36de42ee4d9..e2ff8ea1cb79 100644
--- a/tools/testing/selftests/bpf/benchs/bench_local_storage_create.c
+++ b/tools/testing/selftests/bpf/benchs/bench_local_storage_create.c
@@ -186,7 +186,7 @@ static void *task_producer(void *input)

for (i = 0; i < batch_sz; i++) {
if (!pthd_results[i])
- pthread_join(pthds[i], NULL);;
+ pthread_join(pthds[i], NULL);
}
}

diff --git a/tools/testing/selftests/bpf/progs/iters.c b/tools/testing/selftests/bpf/progs/iters.c
index 3db416606f2f..fe65e0952a1e 100644
--- a/tools/testing/selftests/bpf/progs/iters.c
+++ b/tools/testing/selftests/bpf/progs/iters.c
@@ -673,7 +673,7 @@ static __noinline void fill(struct bpf_iter_num *it, int *arr, __u32 n, int mul)

static __noinline int sum(struct bpf_iter_num *it, int *arr, __u32 n)
{
- int *t, i, sum = 0;;
+ int *t, i, sum = 0;

while ((t = bpf_iter_num_next(it))) {
i = *t;
--
2.39.2



2024-03-15 14:00:47

by patchwork-bot+netdevbpf

[permalink] [raw]
Subject: Re: [PATCH][next] selftests/bpf: Remove second semicolon

Hello:

This patch was applied to bpf/bpf-next.git (master)
by Daniel Borkmann <[email protected]>:

On Fri, 15 Mar 2024 09:26:54 +0000 you wrote:
> There are statements with two semicolons. Remove the second one, it
> is redundant.
>
> Signed-off-by: Colin Ian King <[email protected]>
> ---
> tools/testing/selftests/bpf/benchs/bench_local_storage_create.c | 2 +-
> tools/testing/selftests/bpf/progs/iters.c | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)

Here is the summary with links:
- [next] selftests/bpf: Remove second semicolon
https://git.kernel.org/bpf/bpf-next/c/4c8644f86c85

You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html