[PATCH] signal: use sizeof() instead of sizeof

[PATCH] signal: use sizeof() instead of sizeof

[Wei Tang]

This patch fixes the checkpatch.pl warning to signal.c:

WARNING: sizeof info should be sizeof(info)

Signed-off-by: Wei Tang <tangwei@cmss.chinamobile.com>
---
 kernel/signal.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/kernel/signal.c b/kernel/signal.c
index f3f1f7a..13b267a 100644
--- a/kernel/signal.c
+++ b/kernel/signal.c
@@ -1900,7 +1900,7 @@ static void ptrace_do_notify(int signr, int exit_code, int why)
 {
 	siginfo_t info;
 
-	memset(&info, 0, sizeof info);
+	memset(&info, 0, sizeof(info));
 	info.si_signo = signr;
 	info.si_code = exit_code;
 	info.si_pid = task_pid_vnr(current);
-- 
1.9.1

Re: [PATCH] signal: use sizeof() instead of sizeof

[Richard Weinberger]

Hi!

Am 02.02.2016 um 08:35 schrieb Wei Tang:
> This patch fixes the checkpatch.pl warning to signal.c:
> 
> WARNING: sizeof info should be sizeof(info)

And why is this patch needed? What problem does it fix?
Complex expressions should be within parents
but in this case it is IMHO perfectly fine.

Thanks,
//richard

Re: 答复: [PATCH] signal: use sizeof() instead of sizeof

[Richard Weinberger]

Am 02.02.2016 um 09:08 schrieb Wei Tang:
> This is a simple patch, just to clear up the output from checkpatch.pl
> script, otherwise, we will get WARNING always.

This does not answer my question.
What problem does it fix? How does your patch make the kernel better?

Doing patches just because of checkpatch warnings (not errors)
is IMHO not good.
It is a script and not human, it does not understand all the corner cases.
All it can do is giving us hints. But this does not mean that you have to
blindly follow them.

Thanks,
//richard