[SSSD] [PATCH] Remove trailing whitespace

Lukas Slebodnik lslebodn at redhat.com
Mon Aug 10 13:00:43 UTC 2015


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



More information about the sssd-devel mailing list