[PATCH] some cleanups for smbldap.c
Alexander Bokovoy
ab at samba.org
Wed Apr 19 14:41:48 UTC 2017
On ke, 19 huhti 2017, vl--- via samba-technical wrote:
> Hi!
>
> The main focus is to make "struct smbldap_state" private to smbldap.c
> for better encapsulation.
>
> Review appreciated!
Looks good to me. This will break ipasam in FreeIPA but I already
started moving it to own private struct as 'struct ldapsam_privates' is
not accessible anymore and I neeed anyway to implement some more of a
backend code that needs own proper private struct. Updating it to use
your wrappers is not a problem.
Please do not remove getter/setter for paged results, this is useful.
RB+.
--
/ Alexander Bokovoy
More information about the samba-technical
mailing list