[SSSD] [INI][PATCHES] All of them (C-Style Comments, Unicode conversion, Space processing, UNIT tests)

Dmitri Pal dpal at redhat.com
Fri Jan 10 17:23:16 UTC 2014


On 01/09/2014 05:57 AM, Lukas Slebodnik wrote:
> I was waiting for output from static analysers. (Coverity was overloaded)
> Warning is gone (solution with variable "initialized" is good enough)
> I don't like very long functions like common_file_convert, but it can be
> refactored in the future.
>
> Thank you for your time.
>
> ACK to all patches
>
> LS
Me too. I will see how it can be re-factored. I will file a ticket for that.
https://fedorahosted.org/sssd/ticket/2192

But is it good enough for now.

-- 
Thank you,
Dmitri Pal

Sr. Engineering Manager for IdM portfolio
Red Hat Inc.


-------------------------------
Looking to carve out IT costs?
www.redhat.com/carveoutcosts/






More information about the sssd-devel mailing list