Full_Name: SATOH Fumiyasu
Version: master, 2.4.31
Submission from: (NULL) (22.214.171.124)
This patch adds missing function declarations to *.h, and make some
functions and a global variable static.
Thanks for the patch. In reviewing it, I see no reason to publish
ldap_pvt_sasl_getmechs or ldap_pvt_tls_check_hostname since neither is ever
referenced outside their defining source file. It looks like your addition of
ldap_create_session_tracking_control is also a mistake, you should simply have
fixed the declaration ldap_create_session_tracking (which is a nonexistent name).
Did you have a particular need to use these functions? Aside from the
ldap_create_session_tracking_control typo I don't think it's worth the trouble
to tweak the others.
-- Howard Chu
CTO, Symas Corp. http://www.symas.com
Director, Highland Sun http://highlandsun.com/hyc/
Chief Architect, OpenLDAP http://www.openldap.org/project/