Clear serverlist prior to rehash

Found by: tabb

Because of the new addserver command (instead of old 'set servers {} ), every time a bot is rehashed, the config file is re-read and addserver would again add the already-added servers to the bot's internal server list, causing duplicates to appear.

src/mod/server.mod/server.c

@@ -1900,7 +1900,14 @@ static void server_5minutely()
 
 static void server_prerehash()
 {
+  struct server_list *x = serverlist;
+  char port[7];
+
   strlcpy(oldnick, botname, sizeof oldnick);
+  for (; x; x = x->next) {
+    egg_snprintf(port, sizeof port, "%d", x->port);
+    del_server(x->name, port);
+  }
 }
 
 static void server_postrehash()

src/mod/server.mod/server.h

@@ -85,8 +85,8 @@
 #define exclusive_binds (*(int *)(server_funcs[39]))
 /* 40 - 43 */
 #define H_out (*(p_tcl_bind_list *)(server_funcs[40]))
-#define addserver ((void(*)(char *))server_funcs[41])
-#define delserver ((void(*)(char *))server_funcs[42])
+/* Was (briefly!) addserver */
+/* Was (briefly!) delserver */
 #define net_type_int (*(int *)(server_funcs[43]))
 #endif /* MAKING_SERVER */