From: | Denis Vlasenko <vda@ilport.com.ua> |
To: | krb5-bugs@mit.edu |
Date: | Tue, 6 Dec 2005 17:19:57 +0200 |
Subject: | local array of structures not declared static |
krb5-1.4.3/src/lib/kadm5/logger.c:
            /*             * If there is a facility present, then parse that.             */            if (cp2) {              const struct {                const char *name;                int value;              } facilities[] = {                { "AUTH",    LOG_AUTH     },#ifdef  LOG_AUTHPRIV                { "AUTHPRIV",  LOG_AUTHPRIV   },#endif  /* LOG_AUTHPRIV */#ifdef  LOG_KERN                { "KERN",    LOG_KERN     },#endif  /* LOG_KERN */
Shouldn't we add "static" before "const" here?--vda
            /*             * If there is a facility present, then parse that.             */            if (cp2) {              const struct {                const char *name;                int value;              } facilities[] = {                { "AUTH",    LOG_AUTH     },#ifdef  LOG_AUTHPRIV                { "AUTHPRIV",  LOG_AUTHPRIV   },#endif  /* LOG_AUTHPRIV */#ifdef  LOG_KERN                { "KERN",    LOG_KERN     },#endif  /* LOG_KERN */
Shouldn't we add "static" before "const" here?--vda