On Sun, Oct 19, 2014 at 11:48:48AM +0300, Shlomi Fish wrote:
Do you think it would be a good idea to write a test to test for such trailing
space and remove what's left? This test could have some whitelisting of such
files.
Do you think it would be a good idea to write a test to test for such trailing
space and remove what's left? This test could have some whitelisting of such
files.
significant, as explained at
https://rt.perl.org/Ticket/Display.html?id=119599#txn-1251209
Otherwise, whitespace patches should be treated just like any other
patch -- evaluated on its merits. It does seem like a waste of resources
to be submitting tickets and patches that *only* address whitespace, so
personally I'd only make such changes when accompanied (in a separate
commit, of course) with other patches to related code/documentation.
