[SSSD] [PATCH] confdb: Remove unused function confdb_get_long

Pavel Reichl preichl at redhat.com
Tue Sep 29 06:45:02 UTC 2015



On 09/29/2015 08:31 AM, Lukas Slebodnik wrote:
> On (27/09/15 12:49), Pavel Reichl wrote:
>> Hello, please see trivial patch attached.
>
>>From b9f938087973444f0ec26fc24ad68dca7ac63034 Mon Sep 17 00:00:00 2001
>> From: Pavel Reichl <reichl.pavel at gmail.com>
>> Date: Sun, 27 Sep 2015 12:34:20 +0200
>> Subject: [PATCH] confdb: Remove unused function confdb_get_long
>>
>> ---
>> src/confdb/confdb.c | 51 ---------------------------------------------------
>> 1 file changed, 51 deletions(-)
>>
>> diff --git a/src/confdb/confdb.c b/src/confdb/confdb.c
>> index c097aad7745eda4fff051c7da027776f95db0f03..eebd478f74041d2050df9edc283df43a65462340 100644
>> --- a/src/confdb/confdb.c
>> +++ b/src/confdb/confdb.c
>> @@ -475,57 +475,6 @@ failed:
>>      return ret;
>> }
>>
>> -long confdb_get_long(struct confdb_ctx *cdb,
>> -                     const char *section, const char *attribute,
>> -                     long defval, long *result)
>> -{
> Would it be better to consider this function as confdb API
> and add to src/confdb/confdb.h?

It could be.

> It might be useful in the future.

I thought that out policy towards unused functions were to remove them. We did so in commit 2b94ab415b30861f42b68725d9231905baf8c3bd
We can always resurrect removed code if needed.

I'm not too passionate about this I just think we should be consistent.

>
> LS
> _______________________________________________
> sssd-devel mailing list
> sssd-devel at lists.fedorahosted.org
> https://lists.fedorahosted.org/mailman/listinfo/sssd-devel
>


More information about the sssd-devel mailing list