On 08/10/2015 03:00 PM, Lukas Slebodnik wrote:
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
_________
Please see updated patch set attached.