Re: [PATCH v2 2/3] Staging: lustre: fix braces are not necessary in dt_object.c

From: Joe Perches
Date: Wed Jun 10 2015 - 16:11:50 EST


On Wed, 2015-06-10 at 22:00 +0200, Zoltán Lajos Kis wrote:
> Fixes a braces {} are not necessary for any arm of this statement
> warning in lustre/lustre/obdclass/dt_object.c that was found by
> the checkpatch.pl tool.
>
> Signed-off-by: Zoltán Lajos Kis <zoltan.lajos.kis@xxxxxxxxx>
> ---
> drivers/staging/lustre/lustre/obdclass/dt_object.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/staging/lustre/lustre/obdclass/dt_object.c b/drivers/staging/lustre/lustre/obdclass/dt_object.c
> index 4b5c828d..46fbf21 100644
> --- a/drivers/staging/lustre/lustre/obdclass/dt_object.c
> +++ b/drivers/staging/lustre/lustre/obdclass/dt_object.c
> @@ -329,9 +329,9 @@ static struct dt_object *dt_reg_open(const struct lu_env *env,
> int result;
>
> result = dt_lookup_dir(env, p, name, fid);
> - if (result == 0){
> + if (result == 0)
> o = dt_locate(env, dt, fid);
> - } else
> + else
> o = ERR_PTR(result);
>
> return o;

My preference for this sort of thing is generally:

static struct dt_object *dt_reg_open(const struct lu_env *env,
struct dt_device *dt,
struct dt_object *p,
const char *name,
struct lu_fid *fid)
{
int result;

result = dt_lookup_dir(env, p, name, fid);
if (result)
return ERR_PTR(result);

return dt_locate(env, dt, fid);
}


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/