refactoring more of goodies.h into strings.h

src/kernel/region.c

@@ -42,7 +42,7 @@ OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
 /* util includes */
 #include <util/attrib.h>
 #include <util/bsdstring.h>
-#include <util/goodies.h>
+#include <util/strings.h>
 #include <util/lists.h>
 #include <util/log.h>
 #include <util/resolve.h>

src/kernel/spell.c

@@ -22,7 +22,7 @@ OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
 
 /* util includes */
 #include <critbit.h>
-#include <util/goodies.h>
+#include <util/strings.h>
 #include <util/language.h>
 #include <util/log.h>
 #include <util/umlaut.h>

src/kernel/unit.c

@@ -48,7 +48,7 @@ OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
 #include <util/base36.h>
 #include <util/bsdstring.h>
 #include <util/event.h>
-#include <util/goodies.h>
+#include <util/strings.h>
 #include <util/language.h>
 #include <util/lists.h>
 #include <util/log.h>

src/modules/arena.c

@@ -49,7 +49,7 @@ OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
 #include <util/base36.h>
 #include <util/event.h>
 #include <util/functions.h>
-#include <util/goodies.h>
+#include <util/strings.h>
 #include <util/lists.h>
 #include <util/log.h>
 #include <util/resolve.h>

src/modules/museum.c

@@ -43,7 +43,7 @@ OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
 #include <util/attrib.h>
 #include <util/base36.h>
 #include <util/functions.h>
-#include <util/goodies.h>
+#include <util/strings.h>
 #include <util/language.h>
 
 #include <storage.h>

src/util/crmessage.c

@@ -15,7 +15,7 @@
 #include "crmessage.h"
 
 #include "message.h"
-#include "goodies.h"
+#include "strings.h"
 #include "log.h"
 
 #include <stdio.h>

src/util/goodies.c

@@ -19,12 +19,8 @@ OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
 #include <platform.h>
 #include "goodies.h"
 
-#include "unicode.h"
-
 /* libc includes */
-#include <wctype.h>
 #include <stdlib.h>
-#include <string.h>
 
 /* Simple Integer-Liste */
 
@@ -53,23 +49,6 @@ int *intlist_find(int *i_p, int fi)
     return NULL;
 }
 
-char *set_string(char **s, const char *neu)
-{
-    if (neu == NULL) {
-        free(*s);
-        *s = NULL;
-    }
-    else if (*s == NULL) {
-        *s = malloc(strlen(neu) + 1);
-        strcpy(*s, neu);
-    }
-    else {
-        *s = realloc(*s, strlen(neu) + 1);
-        strcpy(*s, neu);
-    }
-    return *s;
-}
-
 static int spc_email_isvalid(const char *address)
 {
     int count = 0;

src/util/goodies.h

@@ -19,18 +19,15 @@ OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
 #ifndef GOODIES_H
 #define GOODIES_H
 
-#include "strings.h"
-
 #ifdef __cplusplus
 extern "C" {
 #endif
 
-    extern char *set_string(char **s, const char *neu);
+    int set_email(char **pemail, const char *newmail);
-    extern int set_email(char **pemail, const char *newmail);
 
-    extern int *intlist_init(void);
+    int *intlist_init(void);
-    extern int *intlist_add(int *i_p, int i);
+    int *intlist_add(int *i_p, int i);
-    extern int *intlist_find(int *i_p, int i);
+    int *intlist_find(int *i_p, int i);
 
 #ifdef __cplusplus
 }

src/util/language.c

@@ -21,7 +21,7 @@ OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
 #include "language_struct.h"
 
 #include "log.h"
-#include "goodies.h"
+#include "strings.h"
 #include "umlaut.h"
 
 #include <stdlib.h>

src/util/message.c

@@ -14,7 +14,7 @@
 #include <platform.h>
 #include "message.h"
 
-#include "goodies.h"
+#include "strings.h"
 #include "log.h"
 #include "quicklist.h"
 

src/util/nrmessage.c

@@ -21,7 +21,7 @@
 #include "message.h"
 #include "language.h"
 #include "translation.h"
-#include "goodies.h"
+#include "strings.h"
 
 /* libc includes */
 #include <assert.h>

src/util/strings.c

@@ -21,6 +21,24 @@ OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
 /* libc includes */
 #include <string.h>
 #include <assert.h>
+#include <stdlib.h>
+
+char *set_string(char **s, const char *neu)
+{
+    if (neu == NULL) {
+        free(*s);
+        *s = NULL;
+    }
+    else if (*s == NULL) {
+        *s = malloc(strlen(neu) + 1);
+        strcpy(*s, neu);
+    }
+    else {
+        *s = realloc(*s, strlen(neu) + 1);
+        strcpy(*s, neu);
+    }
+    return *s;
+}
 
 unsigned int hashstring(const char *s)
 {

src/util/strings.h

@@ -22,11 +22,11 @@ OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
 extern "C" {
 #endif
 
-    extern unsigned int hashstring(const char *s);
+    char *set_string(char **s, const char *neu);
-    extern const char *escape_string(const char *str, char *buffer,
+    unsigned int hashstring(const char *s);
-        size_t len);
+    const char *escape_string(const char *str, char *buffer, size_t len);
-    extern unsigned int jenkins_hash(unsigned int a);
+    unsigned int jenkins_hash(unsigned int a);
-    extern unsigned int wang_hash(unsigned int a);
+    unsigned int wang_hash(unsigned int a);
 
     /* benchmark for units:
      * JENKINS_HASH: 5.25 misses/hit (with good cache behavior)