[webkit-dev] Removal of trailing whitespace

Chris Dumez cdumez at apple.com
Wed Apr 12 09:17:49 PDT 2023


I am against this because it adds a lot of noise to patches I am trying to review.
I have seen PRs where white space changes account for more than half the patch I am trying to review.

Dropping trailing spaces on the lines you’re modifying is OK but in the whole file is too noisy IMO.

Chris.

> On Apr 12, 2023, at 1:22 AM, Anne van Kesteren via webkit-dev <webkit-dev at lists.webkit.org> wrote:
> 
> To reduce the overhead of switching between projects with different
> whitespace requirements, I would like to suggest we start being
> lenient when trailing whitespace is removed. In particular when a file
> is being changed to fix a bug.
> 
> I could see going even further and enforcing this via the style
> checker, if there is appetite for that.
> 
> Thanks for considering!
> _______________________________________________
> webkit-dev mailing list
> webkit-dev at lists.webkit.org
> https://lists.webkit.org/mailman/listinfo/webkit-dev

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-dev/attachments/20230412/03d22816/attachment.htm>


More information about the webkit-dev mailing list