Thread (106 messages) 106 messages, 12 authors, 2016-10-28

Re: [PATCH 37/54] md/raid5: Replace a seq_printf() call by seq_puts() in raid5_status()

From: SF Markus Elfring <hidden>
Date: 2016-10-06 17:50:26
Also in: kernel-janitors, lkml

quoted
quoted
quoted
diff --git a/drivers/md/raid5.c b/drivers/md/raid5.c
[]
quoted
@@ -7044,7 +7044,7 @@ static void raid5_status(struct seq_file *seq, struct mddev *mddev)
 			   rdev && test_bit(In_sync, &rdev->flags) ? "U" : "_");
 	}
 	rcu_read_unlock();
-	seq_printf (seq, "]");
+	seq_puts(seq, "]");
seq_putc
How do you think about the possibility that the script "checkpatch.pl" can also point
such a source code transformation out directly?
Why don't _you_ try to implement that in checkpatch instead?
How are the chances that any other software developer would be quicker (than me) for such
an addition because of more practical knowledge for the programming language "Perl"?

Regards,
Markus
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help