[email protected] wrote: > Full_Name: SATOH Fumiyasu > Version: master, 2.4.31 > OS: > URL: > ftp://ftp.openldap.org/incoming/openldap-2.4.31-function-declarations.patch > Submission from: (NULL) (220.100.28.128) > > > 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/
