[SSSD] handling NULL params in sss_parse_name

Lukas Slebodnik lslebodn at redhat.com
Mon Jan 27 13:25:25 UTC 2014


On (27/01/14 13:43), Pavel Reichl wrote:
>[snip]
>> 
>> Can you also add '_' as prefix for output parameters into declaration.
>> We are not consistent in this and I cannot find this in coding style,
>> but it improves readability.
>> 
>> LS
>
>Hello Lukas,
>
>thank you for your comment. New patch implementing your suggestion is
>attached.
>
>PR
>
>

>From 95560c8b33ad6b8efdce67b6098d5cdd88fb4129 Mon Sep 17 00:00:00 2001
>From: Pavel Reichl <preichl at redhat.com>
>Date: Sun, 26 Jan 2014 12:39:43 +0000
>Subject: [PATCH] utils: handling NULL params in sss_parse_name
>
>---
> src/util/usertools.c | 50 +++++++++++++++++++++++++++-----------------------
> src/util/util.h      |  2 +-
> 2 files changed, 28 insertions(+), 24 deletions(-)

ACK

LS



More information about the sssd-devel mailing list