From f9db3ac14855eb07e49f2f5797cbb7d338bb614b Mon Sep 17 00:00:00 2001 From: Rich Megginson Date: Tue, 19 May 2009 13:17:11 -0600 Subject: Fix various compiler warnings 1) Make sure we use "const" consistently 2) Make sure we use "unsigned char" consistently for some reason (unsigned char)*p did not compare to '\xHH' literals unless the literal was also cast to (unsigned char) 3) added some missing function prototypes 4) removed some unused variables/functions, or commented out for use when debugging 5) various other compiler warnings With all of these, the code compiles cleanly on RHEL5 x86_64 using gcc (GCC) 4.1.2 20080704 (Red Hat 4.1.2-44) and CFLAGS="-O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector --param=ssp-buffer-size=4 -m64 -mtune=generic" The only warning now is the spurious message about %llu or %lld having the wrong format argument. Reviewed by: nhosoi (Thanks!) --- ldap/servers/slapd/libglobs.c | 6 ------ 1 file changed, 6 deletions(-) (limited to 'ldap/servers/slapd/libglobs.c') diff --git a/ldap/servers/slapd/libglobs.c b/ldap/servers/slapd/libglobs.c index 8c13a9b6..1155c8c7 100644 --- a/ldap/servers/slapd/libglobs.c +++ b/ldap/servers/slapd/libglobs.c @@ -124,12 +124,6 @@ static int config_set_onoff( const char *attrname, char *value, static int config_set_schemareplace ( const char *attrname, char *value, char *errorbuf, int apply ); -static int -isIntegralType(ConfigVarType type) -{ - return type == CONFIG_INT || type == CONFIG_LONG || type == CONFIG_ON_OFF; -} - static int isInt(ConfigVarType type) { -- cgit