Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934047AbbHDNee (ORCPT ); Tue, 4 Aug 2015 09:34:34 -0400 Received: from mail3-relais-sop.national.inria.fr ([192.134.164.104]:63479 "EHLO mail3-relais-sop.national.inria.fr" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933077AbbHDNed (ORCPT ); Tue, 4 Aug 2015 09:34:33 -0400 X-IronPort-AV: E=Sophos;i="5.15,608,1432591200"; d="scan'208";a="142000374" Date: Tue, 4 Aug 2015 15:34:27 +0200 (CEST) From: Julia Lawall X-X-Sender: jll@hadrien To: Shraddha Barke cc: linux-kernel@vger.kernel.org, Julia Lawall , Joe Perches , Jiayi Ye , Greg Kroah-Hartman , Andreas Dilger , Oleg Drokin , devel@driverdev.osuosl.org, HPDD-discuss@ml01.01.org Subject: Re: [PATCH v2 2/3] Staging: lustre: Replace comma between expression statements by a semicolon. In-Reply-To: <1438693573-5310-2-git-send-email-shraddha.6596@gmail.com> Message-ID: References: <1438693573-5310-1-git-send-email-shraddha.6596@gmail.com> <1438693573-5310-2-git-send-email-shraddha.6596@gmail.com> User-Agent: Alpine 2.10 (DEB 1266 2009-07-14) MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1467 Lines: 47 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@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/