Re: [PATCH] checkpatch.pl: Remove --file option

From: Lars-Peter Clausen
Date: Fri Jul 18 2014 - 10:17:46 EST


On 07/17/2014 05:34 PM, Richard Weinberger wrote:
[...]
In order to deal with that bad habit let's remove the --file option
and bring checkpatch.pl back to its original purpose.

I don't think this is a good solution the problem and I'm not sure how successful it will actually be at fixing the problem. How about instead properly documenting that single line trivial style cleanup patches are not really helpful and people should not send them? (Maybe add a check to checkpatch.pl that emits a warning if the patch is a trivial style cleanup ;))

--file is a in my opinion useful option and at least I use it on a regular basis.

- Lars
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/