[dm-devel] [PATCH 1/6] multipathd/main.c: Fix indentation
Martin Wilck
mwilck at suse.com
Mon Mar 5 16:09:56 UTC 2018
On Thu, 2018-03-01 at 11:29 -0800, Bart Van Assche wrote:
> This patch avoids that gcc reports the following:
>
> main.c: In function 'uev_pathfail_check':
> main.c:1022:5: warning: this 'if' clause does not guard... [-
> Wmisleading-indentation]
> if (!pp)
> ^~
> main.c:1024:2: note: ...this statement, but the latter is
> misleadingly indented as if it were guarded by the 'if'
> r = io_err_stat_handle_pathfail(pp);
> ^
>
> Signed-off-by: Bart Van Assche <bart.vanassche at wdc.com>
Reviewed-by: Martin Wilck <mwilck at suse.com>
> ---
> multipathd/main.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/multipathd/main.c b/multipathd/main.c
> index 27cf234623d0..ccbb0ad13d32 100644
> --- a/multipathd/main.c
> +++ b/multipathd/main.c
> @@ -1020,8 +1020,8 @@ uev_pathfail_check(struct uevent *uev, struct
> vectors *vecs)
> lock(&vecs->lock);
> pthread_testcancel();
> pp = find_path_by_devt(vecs->pathvec, devt);
> - if (!pp)
> - goto out_lock;
> + if (!pp)
> + goto out_lock;
> r = io_err_stat_handle_pathfail(pp);
> if (r)
> condlog(3, "io_err_stat: %s: cannot handle pathfail
> uevent",
--
Dr. Martin Wilck <mwilck at suse.com>, Tel. +49 (0)911 74053 2107
SUSE Linux GmbH, GF: Felix Imendörffer, Jane Smithard, Graham Norton
HRB 21284 (AG Nürnberg)
More information about the dm-devel
mailing list