On (10/08/15 14:54), Pavel Reichl wrote:
On 08/10/2015 02:52 PM, Lukas Slebodnik wrote:
>On (10/08/15 14:44), Pavel Reichl wrote:
>>Hello,
>>
>>please see trivial patch attached. I noticed some whitespace in source file,
>>so I did some grepping and decided to remove it as we already were almost 100
>>% trailing whitespace free.
>>
>I would prefer if you could write a test.
>So it will be checked as a part of "make check"
Yes, I already put on my todo list, but I consider it as independent task.
Yes,
but patch is not very useful without the test.
Because we can still accidentally add trailing white spaces.
I would rather wait until you write the test then we can push them
together.
Conditional NACK
LS