[core] remove some unused header includes

remove exposure of stdio.h in buffer.h for print_backtrace(), now static

src/buffer.c

@@ -5,10 +5,6 @@
 #include <stdlib.h>
 #include <string.h>
 
-#include <stdio.h>
-#include <assert.h>
-#include <ctype.h>
-
 static const char hex_chars[] = "0123456789abcdef";
 
 /**
@@ -1006,11 +1002,14 @@ void buffer_to_upper(buffer *b) {
 	}
 }
 
+
+#include <stdio.h>
+
 #ifdef HAVE_LIBUNWIND
 # define UNW_LOCAL_ONLY
 # include <libunwind.h>
 
-void print_backtrace(FILE *file) {
+static void print_backtrace(FILE *file) {
 	unw_cursor_t cursor;
 	unw_context_t context;
 	int ret;
@@ -1071,7 +1070,7 @@ error:
 	fprintf(file, "Error while generating backtrace: unwind error %i\n", (int) -ret);
 }
 #else
-void print_backtrace(FILE *file) {
+static void print_backtrace(FILE *file) {
 	UNUSED(file);
 }
 #endif

src/buffer.h

@@ -4,9 +4,7 @@
 
 #include "settings.h"
 
-#include <stdlib.h>
 #include <sys/types.h>
-#include <stdio.h>
 #include <time.h>
 
 #if defined HAVE_STDINT_H
@@ -163,7 +161,6 @@ static inline void buffer_append_slash(buffer *b); /* append '/' no non-empty st
 #define CONST_BUF_LEN(x) ((x) ? (x)->ptr : NULL), buffer_string_length(x)
 
 
-void print_backtrace(FILE *file);
 void log_failed_assert(const char *filename, unsigned int line, const char *msg) LI_NORETURN;
 #define force_assert(x) do { if (!(x)) log_failed_assert(__FILE__, __LINE__, "assertion failed: " #x); } while(0)
 #define SEGFAULT() log_failed_assert(__FILE__, __LINE__, "aborted");

src/chunk.c

@@ -18,7 +18,6 @@
 #include <fcntl.h>
 #include <unistd.h>
 
-#include <stdio.h>
 #include <errno.h>
 #include <string.h>
 

src/configfile.c

@@ -20,7 +20,6 @@
 #include <stdio.h>
 #include <ctype.h>
 #include <limits.h>
-#include <assert.h>
 #include <glob.h>
 
 

src/configparser.y

@@ -9,7 +9,6 @@
 #include "array.h"
 #include "request.h" /* http_request_host_normalize() */
 
-#include <assert.h>
 #include <ctype.h>
 #include <errno.h>
 #include <stdio.h>

src/connections.c

@@ -21,12 +21,9 @@
 #include <sys/stat.h>
 
 #include <stdlib.h>
-#include <stdio.h>
 #include <unistd.h>
 #include <errno.h>
 #include <string.h>
-#include <fcntl.h>
-#include <assert.h>
 
 #ifdef HAVE_SYS_FILIO_H
 # include <sys/filio.h>

src/data_string.c

@@ -5,7 +5,6 @@
 #include <string.h>
 #include <stdio.h>
 #include <stdlib.h>
-#include <assert.h>
 
 static data_unset *data_string_copy(const data_unset *s) {
 	data_string *src = (data_string *)s;

src/fdevent.c

@@ -9,9 +9,7 @@
 #include <stdlib.h>
 #include <string.h>
 #include <errno.h>
-#include <stdio.h>
 #include <fcntl.h>
-#include <assert.h>
 
 
 fdevents *fdevent_init(server *srv, size_t maxfds, fdevent_handler_t type) {

src/fdevent_freebsd_kqueue.c

@@ -8,10 +8,8 @@
 
 #include <unistd.h>
 #include <stdlib.h>
-#include <stdio.h>
 #include <string.h>
 #include <errno.h>
-#include <signal.h>
 #include <fcntl.h>
 
 #ifdef USE_FREEBSD_KQUEUE

src/fdevent_libev.c

@@ -4,8 +4,6 @@
 #include "buffer.h"
 #include "log.h"
 
-#include <assert.h>
-
 #ifdef USE_LIBEV
 
 # include <ev.h>

src/fdevent_linux_sysepoll.c

@@ -8,11 +8,8 @@
 
 #include <unistd.h>
 #include <stdlib.h>
-#include <stdio.h>
 #include <string.h>
 #include <errno.h>
-#include <signal.h>
-#include <fcntl.h>
 
 #ifdef USE_LINUX_EPOLL
 

src/fdevent_poll.c

@@ -8,11 +8,8 @@
 
 #include <unistd.h>
 #include <stdlib.h>
-#include <stdio.h>
 #include <string.h>
 #include <errno.h>
-#include <signal.h>
-#include <fcntl.h>
 
 #ifdef USE_POLL
 

src/fdevent_select.c

@@ -11,9 +11,6 @@
 #include <stdlib.h>
 #include <string.h>
 #include <errno.h>
-#include <signal.h>
-#include <fcntl.h>
-#include <assert.h>
 
 #ifdef USE_SELECT
 

src/fdevent_solaris_devpoll.c

@@ -8,10 +8,8 @@
 
 #include <unistd.h>
 #include <stdlib.h>
-#include <stdio.h>
 #include <string.h>
 #include <errno.h>
-#include <signal.h>
 #include <fcntl.h>
 
 #ifdef USE_SOLARIS_DEVPOLL

src/fdevent_solaris_port.c

@@ -8,10 +8,8 @@
 
 #include <unistd.h>
 #include <stdlib.h>
-#include <stdio.h>
 #include <string.h>
 #include <errno.h>
-#include <signal.h>
 #include <fcntl.h>
 
 #ifdef USE_SOLARIS_PORT

src/http_chunk.c

@@ -16,10 +16,8 @@
 #include <sys/stat.h>
 
 #include <stdlib.h>
-#include <fcntl.h>
 #include <unistd.h>
 
-#include <stdio.h>
 #include <errno.h>
 #include <string.h>
 

src/keyvalue.c

@@ -6,7 +6,6 @@
 
 #include <stdlib.h>
 #include <string.h>
-#include <stdio.h>
 
 static keyvalue http_versions[] = {
 	{ HTTP_VERSION_1_1, "HTTP/1.1" },

src/md5.h

@@ -27,7 +27,6 @@ These notices must be retained in any copies of any part of this
 documentation and/or software.
  */
 
-#include <limits.h>
 #ifdef HAVE_STDINT_H
 # include <stdint.h>
 #endif

src/mod_access.c

@@ -6,7 +6,6 @@
 
 #include "plugin.h"
 
-#include <ctype.h>
 #include <stdlib.h>
 #include <string.h>
 

src/mod_accesslog.c

@@ -9,7 +9,6 @@
 #include <sys/types.h>
 #include <sys/stat.h>
 
-#include <ctype.h>
 #include <stdlib.h>
 #include <string.h>
 #include <fcntl.h>
@@ -17,8 +16,6 @@
 #include <errno.h>
 #include <time.h>
 
-#include <stdio.h>
-
 #ifdef HAVE_SYSLOG_H
 # include <syslog.h>
 #endif

src/mod_alias.c

@@ -6,10 +6,8 @@
 
 #include "plugin.h"
 
-#include <ctype.h>
 #include <stdlib.h>
 #include <string.h>
-#include <stdio.h>
 
 /* plugin config for all request/connections */
 typedef struct {

src/mod_authn_file.c

@@ -34,7 +34,6 @@
 #include "log.h"
 #include "response.h"
 
-#include "inet_ntop_cache.h"
 #include "base64.h"
 #include "md5.h"
 
@@ -42,6 +41,7 @@
 #include <sys/stat.h>
 
 #include <stdlib.h>
+#include <stdio.h>
 #include <string.h>
 #include <errno.h>
 #include <fcntl.h>

src/mod_authn_ldap.c

@@ -8,7 +8,6 @@
 #include "log.h"
 #include "plugin.h"
 
-#include <ctype.h>
 #include <errno.h>
 #include <string.h>
 

src/mod_authn_mysql.c

@@ -26,8 +26,8 @@
 #include "md5.h"
 #include "plugin.h"
 
-#include <ctype.h>
 #include <errno.h>
+#include <stdio.h>
 #include <string.h>
 
 #ifdef HAVE_CRYPT_H

src/mod_cgi.c

@@ -13,15 +13,8 @@
 
 #include <sys/types.h>
 #include "sys-mmap.h"
-
-#ifdef __WIN32
-# include <winsock2.h>
-#else
-# include <sys/socket.h>
+#include "sys-socket.h"
 # include <sys/wait.h>
-# include <netinet/in.h>
-# include <arpa/inet.h>
-#endif
 
 #include <unistd.h>
 #include <errno.h>
@@ -29,8 +22,6 @@
 #include <string.h>
 #include <fdevent.h>
 #include <signal.h>
-#include <ctype.h>
-#include <assert.h>
 
 #include <stdio.h>
 #include <fcntl.h>

src/mod_cml.c

@@ -15,7 +15,6 @@
 #include <string.h>
 #include <errno.h>
 #include <unistd.h>
-#include <stdio.h>
 
 /* init the plugin data */
 INIT_FUNC(mod_cml_init) {

src/mod_cml_funcs.c

@@ -8,7 +8,6 @@
 #include <errno.h>
 #include <unistd.h>
 #include <dirent.h>
-#include <stdio.h>
 
 #include <lauxlib.h>
 

src/mod_cml_lua.c

@@ -4,8 +4,6 @@
 #include <lualib.h>
 #include <lauxlib.h>
 
-#include <assert.h>
-#include <stdio.h>
 #include <errno.h>
 #include <time.h>
 #include <string.h>

src/mod_compress.c

@@ -14,10 +14,8 @@
 #include <sys/types.h>
 #include <sys/stat.h>
 
-#include <assert.h>
 #include <fcntl.h>
 #include <unistd.h>
-#include <ctype.h>
 #include <stdlib.h>
 #include <string.h>
 #include <strings.h>

src/mod_dirlisting.c

@@ -9,14 +9,11 @@
 #include "response.h"
 #include "stat_cache.h"
 
-#include <ctype.h>
 #include <stdlib.h>
 #include <string.h>
 #include <dirent.h>
-#include <assert.h>
 #include <errno.h>
 #include <fcntl.h>
-#include <stdio.h>
 #include <unistd.h>
 #include <time.h>
 

src/mod_evasive.c

@@ -9,7 +9,6 @@
 
 #include "inet_ntop_cache.h"
 
-#include <ctype.h>
 #include <stdlib.h>
 #include <string.h>
 

src/mod_evhost.c

@@ -7,7 +7,6 @@
 
 #include <string.h>
 #include <errno.h>
-#include <ctype.h>
 
 typedef struct {
 	/* unparsed pieces */

src/mod_expire.c

@@ -8,7 +8,6 @@
 #include "plugin.h"
 #include "stat_cache.h"
 
-#include <ctype.h>
 #include <stdlib.h>
 #include <string.h>
 #include <time.h>

src/mod_extforward.c

@@ -6,15 +6,11 @@
 
 #include "plugin.h"
 
-#include "inet_ntop_cache.h"
 #include "configfile.h"
 
-#include <assert.h>
-#include <ctype.h>
+#include "sys-socket.h"
 #include <stdlib.h>
 #include <string.h>
-#include <stdio.h>
-#include <netinet/in.h>
 #include <errno.h>
 
 /**

src/mod_fastcgi.c

@@ -13,7 +13,6 @@
 
 #include "plugin.h"
 
-#include "inet_ntop_cache.h"
 #include "stat_cache.h"
 #include "status_counter.h"
 
@@ -23,8 +22,6 @@
 #include <fcntl.h>
 #include <string.h>
 #include <stdlib.h>
-#include <ctype.h>
-#include <assert.h>
 #include <signal.h>
 
 #ifdef HAVE_FASTCGI_FASTCGI_H
@@ -37,8 +34,6 @@
 # endif
 #endif /* HAVE_FASTCGI_FASTCGI_H */
 
-#include <stdio.h>
-
 #include "sys-socket.h"
 
 #ifdef HAVE_SYS_UIO_H

src/mod_flv_streaming.c

@@ -9,7 +9,6 @@
 
 #include "plugin.h"
 
-#include <ctype.h>
 #include <stdlib.h>
 #include <string.h>
 

src/mod_indexfile.c

@@ -8,7 +8,6 @@
 
 #include "stat_cache.h"
 
-#include <ctype.h>
 #include <stdlib.h>
 #include <string.h>
 #include <errno.h>

src/mod_magnet.c

@@ -13,10 +13,8 @@
 #include "status_counter.h"
 #include "etag.h"
 
-#include <ctype.h>
 #include <stdlib.h>
 #include <string.h>
-#include <assert.h>
 #include <setjmp.h>
 
 #include <lua.h>

src/mod_magnet_cache.c

@@ -5,7 +5,6 @@
 
 #include <stdlib.h>
 #include <time.h>
-#include <assert.h>
 
 #include <lualib.h>
 #include <lauxlib.h>

src/mod_mysql_vhost.c

@@ -1,9 +1,7 @@
 #include "first.h"
 
 #include <unistd.h>
-#include <stdio.h>
 #include <errno.h>
-#include <fcntl.h>
 #include <string.h>
 
 #include <mysql.h>

src/mod_proxy.c

@@ -20,13 +20,9 @@
 
 #include <unistd.h>
 #include <errno.h>
-#include <fcntl.h>
+#include <limits.h>
 #include <string.h>
 #include <stdlib.h>
-#include <ctype.h>
-#include <assert.h>
-
-#include <stdio.h>
 
 #include "sys-socket.h"
 

src/mod_rrdtool.c

@@ -9,10 +9,8 @@
 #include "plugin.h"
 #include <sys/types.h>
 
-#include <assert.h>
 #include <fcntl.h>
 #include <stdlib.h>
-#include <stdio.h>
 #include <string.h>
 #include <unistd.h>
 #include <errno.h>

src/mod_scgi.c

@@ -13,20 +13,15 @@
 
 #include "plugin.h"
 
-#include "inet_ntop_cache.h"
-
 #include <sys/types.h>
 #include <unistd.h>
 #include <errno.h>
 #include <fcntl.h>
+#include <limits.h>
 #include <string.h>
 #include <stdlib.h>
-#include <ctype.h>
-#include <assert.h>
 #include <signal.h>
 
-#include <stdio.h>
-
 #include "sys-socket.h"
 #include "sys-endian.h"
 

src/mod_secdownload.c

@@ -7,7 +7,6 @@
 
 #include "plugin.h"
 
-#include <ctype.h>
 #include <stdlib.h>
 #include <string.h>
 

src/mod_simple_vhost.c

@@ -7,8 +7,6 @@
 
 #include "plugin.h"
 
-#include <assert.h>
-#include <ctype.h>
 #include <stdlib.h>
 #include <string.h>
 #include <errno.h>

src/mod_skeleton.c

@@ -6,7 +6,6 @@
 
 #include "plugin.h"
 
-#include <ctype.h>
 #include <stdlib.h>
 #include <string.h>
 

src/mod_ssi.c

@@ -10,15 +10,12 @@
 
 #include "mod_ssi.h"
 
-#include "inet_ntop_cache.h"
-
 #include "sys-socket.h"
 
 #include <sys/types.h>
 
 #include <ctype.h>
 #include <stdlib.h>
-#include <stdio.h>
 #include <string.h>
 #include <strings.h>
 #include <errno.h>

src/mod_ssi_exprparser.y

@@ -8,7 +8,6 @@
 #include "mod_ssi_expr.h"
 #include "buffer.h"
 
-#include <assert.h>
 #include <string.h>
 }
 

src/mod_staticfile.c

@@ -11,9 +11,7 @@
 #include "http_chunk.h"
 #include "response.h"
 
-#include <ctype.h>
 #include <stdlib.h>
-#include <stdio.h>
 #include <string.h>
 
 /**

src/mod_trigger_b4_dl.c

@@ -8,9 +8,7 @@
 #include "response.h"
 #include "inet_ntop_cache.h"
 
-#include <ctype.h>
 #include <stdlib.h>
-#include <fcntl.h>
 #include <string.h>
 
 #if defined(HAVE_GDBM_H)

src/mod_usertrack.c

@@ -7,7 +7,6 @@
 
 #include "plugin.h"
 
-#include <ctype.h>
 #include <stdlib.h>
 #include <string.h>
 

src/mod_webdav.c

@@ -14,13 +14,10 @@
 
 #include <sys/types.h>
 #include <sys/stat.h>
-#include <ctype.h>
 #include <stdlib.h>
 #include <string.h>
 #include <errno.h>
 #include <fcntl.h>
-#include <stdio.h>
-#include <assert.h>
 
 #include <unistd.h>
 #include <dirent.h>

src/network.c

@@ -21,7 +21,6 @@
 #include <unistd.h>
 #include <string.h>
 #include <stdlib.h>
-#include <assert.h>
 
 void
 network_accept_tcp_nagle_disable (const int fd)

src/plugin.c

@@ -6,8 +6,6 @@
 #include <string.h>
 #include <stdlib.h>
 
-#include <stdio.h>
-
 #ifdef HAVE_VALGRIND_VALGRIND_H
 # include <valgrind/valgrind.h>
 #endif

src/proc_open.c

@@ -4,7 +4,6 @@
 
 #include <stdlib.h>
 #include <stdio.h>
-#include <ctype.h>
 #include <string.h>
 #include <errno.h>
 

src/rand.c

@@ -9,7 +9,6 @@
 #include <sys/stat.h>
 #include <errno.h>
 #include <fcntl.h>
-#include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
 #include <time.h>

src/request.c

@@ -9,8 +9,6 @@
 #include <limits.h>
 #include <stdlib.h>
 #include <string.h>
-#include <stdio.h>
-#include <ctype.h>
 
 static int request_check_hostname(buffer *host) {
 	enum { DOMAINLABEL, TOPLABEL } stage = TOPLABEL;

src/response.c

@@ -15,15 +15,10 @@
 
 #include <limits.h>
 #include <errno.h>
-#include <fcntl.h>
 #include <stdlib.h>
 #include <string.h>
 #include <time.h>
 #include <unistd.h>
-#include <ctype.h>
-#include <assert.h>
-
-#include <stdio.h>
 
 #include "sys-socket.h"
 

src/server.c

@@ -39,7 +39,6 @@
 #include <stdlib.h>
 #include <time.h>
 #include <signal.h>
-#include <assert.h>
 #include <locale.h>
 
 #include <stdio.h>

src/stat_cache.c

@@ -12,9 +12,7 @@
 #include <string.h>
 #include <errno.h>
 #include <unistd.h>
-#include <stdio.h>
 #include <fcntl.h>
-#include <assert.h>
 
 #ifdef HAVE_ATTR_ATTRIBUTES_H
 # include <attr/attributes.h>

src/stream.c

@@ -7,6 +7,7 @@
 
 #include <unistd.h>
 #include <fcntl.h>
+#include <stdlib.h>
 
 #include "sys-mmap.h"
 

src/test_buffer.c

@@ -1,5 +1,8 @@
 #include "first.h"
 
+#include <stdio.h>
+#include <stdlib.h>
+
 #include "buffer.h"
 
 static void run_buffer_path_simplify(buffer *psrc, buffer *pdest, const char *in, size_t in_len, const char *out, size_t out_len) {