linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH]  Staging : lustre :Replace comma with a semicolon
@ 2015-08-03 15:11 Shraddha Barke
  2015-08-03 15:34 ` Dan Carpenter
  0 siblings, 1 reply; 6+ messages in thread
From: Shraddha Barke @ 2015-08-03 15:11 UTC (permalink / raw)
  To: linux-kernel, Fan Yong, Julia Lawall, Greg Kroah-Hartman,
	Andreas Dilger, Oleg Drokin, devel, HPDD-discuss
  Cc: Shraddha Barke

Replace comma between expression statements by a semicolon.
The semantic patch used is as follows:

@@
expression e1,e2;
@@
e1
- ,
+ ;
e2;

Signed-off-by: Shraddha Barke <shraddha.6596@gmail.com>
---
 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


^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCH]  Staging : lustre :Replace comma with a semicolon
  2015-08-03 15:11 [PATCH] Staging : lustre :Replace comma with a semicolon Shraddha Barke
@ 2015-08-03 15:34 ` Dan Carpenter
       [not found]   ` <CAO393ADR2a2_qB-WFjoBtdXomd_6a3rZMmR=nOxiQCcKLt_R5g@mail.gmail.com>
  0 siblings, 1 reply; 6+ messages in thread
From: Dan Carpenter @ 2015-08-03 15:34 UTC (permalink / raw)
  To: Shraddha Barke
  Cc: linux-kernel, Fan Yong, Julia Lawall, Greg Kroah-Hartman,
	Andreas Dilger, Oleg Drokin, devel, HPDD-discuss

The spacing in the subject lines is whacky.

-Subject: [PATCH]  Staging : lustre :Replace comma with a semicolon
+Subject: [PATCH] Staging: lustre: Replace comma with a semicolon

regards,
dan carpenter


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] Staging : lustre :Replace comma with a semicolon
       [not found]   ` <CAO393ADR2a2_qB-WFjoBtdXomd_6a3rZMmR=nOxiQCcKLt_R5g@mail.gmail.com>
@ 2015-08-03 16:07     ` Greg Kroah-Hartman
  2015-08-04  0:35     ` Greg Kroah-Hartman
  1 sibling, 0 replies; 6+ messages in thread
From: Greg Kroah-Hartman @ 2015-08-03 16:07 UTC (permalink / raw)
  To: Shraddha Barke
  Cc: Dan Carpenter, linux-kernel, Fan Yong, Julia Lawall,
	Andreas Dilger, Oleg Drokin, devel, HPDD-discuss

On Mon, Aug 03, 2015 at 09:22:51PM +0530, Shraddha Barke wrote:
> Should I resend the patches?
> 

Why wouldn't you?

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] Staging : lustre :Replace comma with a semicolon
       [not found]   ` <CAO393ADR2a2_qB-WFjoBtdXomd_6a3rZMmR=nOxiQCcKLt_R5g@mail.gmail.com>
  2015-08-03 16:07     ` Greg Kroah-Hartman
@ 2015-08-04  0:35     ` Greg Kroah-Hartman
  1 sibling, 0 replies; 6+ messages in thread
From: Greg Kroah-Hartman @ 2015-08-04  0:35 UTC (permalink / raw)
  To: Shraddha Barke
  Cc: Dan Carpenter, linux-kernel, Fan Yong, Julia Lawall,
	Andreas Dilger, Oleg Drokin, devel, HPDD-discuss

On Mon, Aug 03, 2015 at 09:22:51PM +0530, Shraddha Barke wrote:
> Should I resend the patches?
> 

You have sent a bunch of patches, and I don't know what order to apply
them in, or what ones should be applied and which should not.

So please resend all of the outstanding patches that I have not applied,
in a patch series, properly numbered, so that I know what order to apply
them in, and how many there are.

thanks,

greg k-h

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH] Staging: lustre: Replace comma with a semicolon
@ 2015-08-03 16:53 Shraddha Barke
  0 siblings, 0 replies; 6+ messages in thread
From: Shraddha Barke @ 2015-08-03 16:53 UTC (permalink / raw)
  To: linux-kernel, Julia Lawall, Bobi Jam, Greg Kroah-Hartman,
	Andreas Dilger, Oleg Drokin, devel
  Cc: Shraddha Barke

Replace comma between expression statements by a semicolon.
The semantic patch used is as follows:

@@
expression e1,e2;
@@
e1
- ,
+ ;
e2;

Signed-off-by: Shraddha Barke <shraddha.6596@gmail.com>
---
 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


^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [PATCH] Staging : lustre :Replace comma with a semicolon
@ 2015-08-03 15:04 Shraddha Barke
  0 siblings, 0 replies; 6+ messages in thread
From: Shraddha Barke @ 2015-08-03 15:04 UTC (permalink / raw)
  To: linux-kernel, Matthew Tyler, Guenter Roeck, Greg Kroah-Hartman,
	Andreas Dilger, Oleg Drokin, devel, HPDD-discuss
  Cc: Shraddha Barke

Replace comma between expression statements by a semicolon.
The semantic patch used is as follows:

@@
expression e1,e2;
@@
e1
- ,
+ ;
e2;

Signed-off-by: Shraddha Barke <shraddha.6596@gmail.com>
---
 drivers/staging/lustre/lustre/libcfs/libcfs_string.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/staging/lustre/lustre/libcfs/libcfs_string.c b/drivers/staging/lustre/lustre/libcfs/libcfs_string.c
index 4dde8e0..efe5e66 100644
--- a/drivers/staging/lustre/lustre/libcfs/libcfs_string.c
+++ b/drivers/staging/lustre/lustre/libcfs/libcfs_string.c
@@ -400,7 +400,7 @@ cfs_expr_list_free(struct cfs_expr_list *expr_list)
 		struct cfs_range_expr *expr;
 
 		expr = list_entry(expr_list->el_exprs.next,
-				      struct cfs_range_expr, re_link),
+				      struct cfs_range_expr, re_link);
 		list_del(&expr->re_link);
 		LIBCFS_FREE(expr, sizeof(*expr));
 	}
-- 
2.1.0


^ permalink raw reply related	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2015-08-04  0:35 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-08-03 15:11 [PATCH] Staging : lustre :Replace comma with a semicolon Shraddha Barke
2015-08-03 15:34 ` Dan Carpenter
     [not found]   ` <CAO393ADR2a2_qB-WFjoBtdXomd_6a3rZMmR=nOxiQCcKLt_R5g@mail.gmail.com>
2015-08-03 16:07     ` Greg Kroah-Hartman
2015-08-04  0:35     ` Greg Kroah-Hartman
  -- strict thread matches above, loose matches on Subject: below --
2015-08-03 16:53 [PATCH] Staging: lustre: Replace " Shraddha Barke
2015-08-03 15:04 [PATCH] Staging : lustre :Replace " Shraddha Barke

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).