[PATCH] some cleanups for smbldap.c

Ralph Böhme slow at samba.org
Wed Apr 19 14:41:36 UTC 2017


On Wed, Apr 19, 2017 at 04:20:36PM +0200, vl--- via samba-technical wrote:
> The main focus is to make "struct smbldap_state" private to smbldap.c
> for better encapsulation.

+1 for the encapsulation but can we possibly use a different name then
smbldap_get_ld() ? The ld makes it look like you could get a linker out of
smbldap. :)

Maybe smbldap_get_ldap() ?

If you don't agree and would like to stick with smbldap_get_ld() please go ahead
and push with my rb.

If you agree but wonder how to rename without revisiting every file, sed inplace
would do the trick, so

$ find . -name '*.c' | xargs sed -i s/smbldap_get_ld/smbldap_get_ldap/g
$ find . -name '*.h' | xargs sed -i s/smbldap_get_ld/smbldap_get_ldap/g

If you want you can offload the task, I can do it, just tell me I should go
ahead and push with this change.

-slow



More information about the samba-technical mailing list