Re: [PATCH] selftests/bpf: Convert comma to semicolon

2025-04-04 Thread patchwork-bot+netdevbpf
Hello: This patch was applied to bpf/bpf-next.git (master) by Andrii Nakryiko : On Tue, 1 Apr 2025 14:15:46 +0800 you wrote: > Replace comma between expressions with semicolons. > > Using a ',' in place of a ';' can have unintended side effects. > Although that is not the case here, it is seems

Re: [PATCH] selftests/bpf: Convert comma to semicolon

2025-04-01 Thread Amery Hung
On Mon, Mar 31, 2025 at 11:17 PM Chen Ni wrote: > > Replace comma between expressions with semicolons. > > Using a ',' in place of a ';' can have unintended side effects. > Although that is not the case here, it is seems best to use ';' > unless ',' is intended. > > Found by inspection. > No funct

[PATCH] selftests/bpf: Convert comma to semicolon

2025-04-01 Thread Chen Ni
Replace comma between expressions with semicolons. Using a ',' in place of a ';' can have unintended side effects. Although that is not the case here, it is seems best to use ';' unless ',' is intended. Found by inspection. No functional change intended. Compile tested only. Signed-off-by: Chen

Re: [PATCH] selftests/bpf: Convert comma to semicolon

2025-03-11 Thread Anton Protopopov
On 25/03/10 11:20AM, Chen Ni wrote: > Replace comma between expressions with semicolons. > > Using a ',' in place of a ';' can have unintended side effects. > Although that is not the case here, it is seems best to use ';' > unless ',' is intended. This is a typo, of course. Thanks! > Found by i

Re: [PATCH] selftests/bpf: Convert comma to semicolon

2025-03-10 Thread patchwork-bot+netdevbpf
Hello: This patch was applied to bpf/bpf-next.git (master) by Andrii Nakryiko : On Mon, 10 Mar 2025 11:20:45 +0800 you wrote: > Replace comma between expressions with semicolons. > > Using a ',' in place of a ';' can have unintended side effects. > Although that is not the case here, it is seems

[PATCH] selftests/bpf: Convert comma to semicolon

2025-03-09 Thread Chen Ni
Replace comma between expressions with semicolons. Using a ',' in place of a ';' can have unintended side effects. Although that is not the case here, it is seems best to use ';' unless ',' is intended. Found by inspection. No functional change intended. Compile tested only. Signed-off-by: Chen