Re: [PATCH] checkpatch: See if modified files are marked obsolete in MAINTAINERS

From: Joe Perches
Date: Mon Aug 22 2016 - 16:56:16 EST


On Mon, 2016-08-22 at 22:50 +0200, SF Markus Elfring wrote:
> > @@ -2289,6 +2299,10 @@ sub process {
> > Â }
> > Â
> > Â if ($found_file) {
> > + if (is_maintained_obsolete($realfile)) {
> > + WARN("OBSOLETE",
> > + ÂÂÂÂÂ"$realfile is marked as 'obsolete' in the MAINTAINERS hierarchy.ÂÂNo unnecessary modifications please.\n");
> > + }
> How do you think about to avoid a double negation in such a warning message?
>
> Would a wording like "â Only really necessary modifications please.\n"
> be more useful here?

No, probably not.