linux-kernel - Re: [PATCH v2 2/3] Staging: lustre: Replace comma between expression statements by a semicolon.

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Tue, 4 Aug 2015 15:34:27 +0200 (CEST)
From:	Julia Lawall <julia.lawall@...6.fr>
To:	Shraddha Barke <shraddha.6596@...il.com>
cc:	linux-kernel@...r.kernel.org, Julia Lawall <Julia.Lawall@...6.fr>,
	Joe Perches <joe@...ches.com>, Jiayi Ye <yejiayily@...il.com>,
	Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
	Andreas Dilger <andreas.dilger@...el.com>,
	Oleg Drokin <oleg.drokin@...el.com>,
	devel@...verdev.osuosl.org, HPDD-discuss@...1.01.org
Subject: Re: [PATCH v2 2/3] Staging: lustre: Replace comma between expression
 statements by a semicolon.

Sorry not to have been more clear.  The original subject text was OK.
Tehway to differentiate between the patches would be to put Staging:
lustre: mgc: in thi case, and Staging: lustre: libcfs: in the other one.

julia

On Tue, 4 Aug 2015, Shraddha Barke wrote:

> Replace comma between expression statements by a semicolon.
> The semantic patch used is as follows:
>
> @@
> expression e1,e2;
> @@
> e1
> - ,
> + ;
> e2;
> ---
> Changes in v2:
>   -Make subject more clearer.
>
>  drivers/staging/lustre/lustre/mgc/mgc_request.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/staging/lustre/lustre/mgc/mgc_request.c b/drivers/staging/lustre/lustre/mgc/mgc_request.c
> index 562bd95..019ee2f 100644
> --- a/drivers/staging/lustre/lustre/mgc/mgc_request.c
> +++ b/drivers/staging/lustre/lustre/mgc/mgc_request.c
> @@ -1232,7 +1232,7 @@ static int mgc_apply_recover_logs(struct obd_device *mgc,
>  		pos += sprintf(obdname + pos, "-%s%04x",
>  				  is_ost ? "OST" : "MDT", entry->mne_index);
>
> -		cname = is_ost ? "osc" : "mdc",
> +		cname = is_ost ? "osc" : "mdc";
>  		pos += sprintf(obdname + pos, "-%s-%s", cname, inst);
>  		lustre_cfg_bufs_reset(&bufs, obdname);
>
> --
> 2.1.0
>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/