On 14/06/2016 10:57, Markus Armbruster wrote: >> diff --git a/scripts/coccinelle/return_directly.cocci >> b/scripts/coccinelle/return_directly.cocci >> > new file mode 100644 >> > index 0000000..c52f4fc >> > --- /dev/null >> > +++ b/scripts/coccinelle/return_directly.cocci >> > @@ -0,0 +1,21 @@ >> > +// replace 'R = X; return R;' with 'return R;' >> > + >> > +// remove assignment > Second comment feels redundant. Can drop on commit to error-next. > >> > +@ removal @ > Rule name "removal" is not used. Can drop on commit to error-next. >
I've seen rule names used as a comment. Feels a bit like COBOL, but it doesn't hurt. Perhaps rename it to "@ return_directly @"? Paolo