Re: [PATCH][next] rv: remove redundant assignment to variable retval
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
- Subject: Re: [PATCH][next] rv: remove redundant assignment to variable retval
- From: Daniel Bristot de Oliveira <bristot@xxxxxxxxxx>
- Date: Mon, 24 Apr 2023 11:20:26 +0200
- Cc: kernel-janitors@xxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, Steven Rostedt <rostedt@xxxxxxxxxxx>, linux-trace-devel@xxxxxxxxxxxxxxx, Masami Hiramatsu <mhiramat@xxxxxxxxxx>
- In-reply-to: <20230418150018.3123753-1-colin.i.king@gmail.com>
- References: <20230418150018.3123753-1-colin.i.king@gmail.com>
- User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.9.1
On 4/18/23 17:00, Colin Ian King wrote:
> Variable retval is being assigned a value that is never read, it is
> being re-assigned a new value in both paths of a following if statement.
> Remove the assignment.
>
> Cleans up clang-scan warning:
> kernel/trace/rv/rv.c:293:2: warning: Value stored to 'retval' is never read [deadcode.DeadStores]
> retval = count;
>
> Signed-off-by: Colin Ian King <colin.i.king@xxxxxxxxx>
Acked-by: Daniel Bristot de Oliveira <bristot@xxxxxxxxxx>
Thanks!
-- Daniel
[Index of Archives]
[Linux USB Development]
[Linux USB Development]
[Linux Audio Users]
[Yosemite Hiking]
[Linux Kernel]
[Linux SCSI]