Hi all! On Thu, 2016-10-06 at 19:49 +0200, SF Markus Elfring wrote: > > diff --git a/drivers/md/raid5.c b/drivers/md/raid5.c > > > > [] > > > > > > > > > > @@ -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"? The above is BTW a pretty simple thing and thus good for a learning experience for regular expressions and copy-pasting a few lines in that perl-script and editing them. MfG, Bernd PS: Sry for the noise - it's somewhat OT here .... -- Bernd Petrovitsch Email : bernd@xxxxxxxxxxxxxxxxxxx LUGA : http://www.luga.at -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html