Messages in this thread | | | Date | Wed, 9 Apr 2008 08:14:12 -0700 | From | Randy Dunlap <> | Subject | Re: [patch] checkpatch: relax spacing and line length |
| |
On Wed, 9 Apr 2008 14:14:20 +0100 Andy Whitcroft wrote:
> === 8< === > add a note on the risks associated with wide ranging style cleanups > > Add a NOTE to the SubmittingPatches guide pointing out the issues which > may occur when submitting wide ranging style cleanups, and pointing the > reader to the area maintainers. > > Signed-off-by: Andy Whitcroft <apw@shadowen.org> > --- > Documentation/SubmittingPatches | 6 ++++++ > 1 files changed, 6 insertions(+), 0 deletions(-) > diff --git a/Documentation/SubmittingPatches b/Documentation/SubmittingPatches > index 08a1ed1..633b9cf 100644 > --- a/Documentation/SubmittingPatches > +++ b/Documentation/SubmittingPatches > @@ -472,6 +472,12 @@ The checker reports at three levels: > You should be able to justify all violations that remain in your > patch. > > +NOTE: updating entire files purely for style violations can cause > +significant issues for other contributers and maintainers by increasing
contributors
> +collisions between patches. It is therefore recommended you contact the > +area Maintainer to ensure these patches are likely to be accepted _before_ > +starting work. > + > > > 2) #ifdefs are ugly
--- ~Randy
| |