Bug 106008 – [style] Re-enable the end-of-line whitespace warnings

• Status: RESOLVED
• Resolution: WONTFIX
• Component: Tools / Tests

[Zan Dobersek]

[style] Reenable the end-of-line whitespace warnings

[Zan Dobersek]

I'm not aware of any syntax that would rely on end-of-line whitespace, so I advocate for enabling that filter (again?).

I personally find this kind of whitespace annoying and not adding anything valuable to the code. I'm also sometimes quite astonished at the amount of it, so I'm also not sure I'm grasping the whole aspect of it - are there some editors forcing this whitespace into the code or is it just a human fault that just (at the moment) goes unnoticed due to the filter being disabled?

[Zan Dobersek]

Created attachment 181161 [details]
Patch

[David Levin]

Comment on attachment 181161 [details]
Patch

This isn't in the style guide and putting up a patch to enforce it isn't really the right way to go about introducing a new style rule.

You're best off bringing it up on webkit-dev. (I would search for past discussions about this. People haven't been too keen on having this warning so I don't expect a warm reception.)

[Zan Dobersek]

Here's the past discussion, for future reference.
https://lists.webkit.org/pipermail/webkit-dev/2009-August/009662.html

I'll probably bring this on the mailing list in future, but in the form of a style guideline change. Closing as a WontFix until then.