Fix header includes to be a bit more consistent

lib/comms.cpp

@@ -1,13 +1,12 @@
 #include "auth.h"
-#include "bitfields.h"
 #include "comms.h"
 #include "defines.h"
-#include "encode.h"
 #include "stream.h"
 #include "procs.h"
 #include "timing.h"
 #include <fcntl.h>
 #include <string.h>
+#include <sstream>
 #include "config.h"
 
 namespace Comms{

lib/config.cpp

@@ -8,6 +8,18 @@
 #include "tinythread.h"
 #include <signal.h>
 #include <string.h>
+#include "procs.h"
+#include <sys/types.h>
+#include <sys/stat.h>
+#include <unistd.h>
+#include <dirent.h> //for getMyExec
+#include <errno.h>
+#include <fcntl.h>
+#include <getopt.h>
+#include <iostream>
+#include <pwd.h>
+#include <stdarg.h> // for va_list
+#include <stdlib.h>
 
 #ifdef __CYGWIN__
 #include <windows.h>
@@ -21,21 +33,6 @@
 #if defined(__APPLE__)
 #include <mach-o/dyld.h>
 #endif
-#include "procs.h"
-#include <dirent.h> //for getMyExec
-#include <errno.h>
-#include <fcntl.h>
-#include <fstream>
-#include <getopt.h>
-#include <iostream>
-#include <pwd.h>
-#include <signal.h>
-#include <string.h>
-#include <stdarg.h> // for va_list
-#include <stdlib.h>
-#include <sys/types.h>
-#include <sys/stat.h>
-#include <unistd.h>
 
 bool Util::Config::is_active = false;
 bool Util::Config::is_restarting = false;

lib/config.h

@@ -7,8 +7,8 @@
 #define PACKAGE_VERSION "unknown"
 #endif
 
+#include <sys/wait.h>
 #include "json.h"
-#include <signal.h>
 #include <string>
 
 /// Contains utility code, not directly related to streaming media

lib/dtsc.cpp

@@ -12,7 +12,7 @@
 #include <cstdlib>
 #include <cstring>
 #include <fstream>
-#include <iomanip>
+#include <sstream>
 
 namespace DTSC{
   char Magic_Header[] = "DTSC";

lib/h264.cpp

@@ -7,6 +7,7 @@
 #include "h264.h"
 #include <cmath>
 #include <cstring>
+#include <sstream>
 #include <iomanip>
 
 namespace h264{

lib/hls_support.cpp

@@ -2,6 +2,7 @@
 #include "langcodes.h" /*LTS*/
 #include "stream.h"
 #include <cstdlib>
+#include <sstream>
 #include <iomanip>
 
 namespace HLS{

lib/http_parser.cpp

@@ -11,6 +11,7 @@
 #include "json.h"
 #include <iomanip>
 #include <strings.h>
+#include <sstream>
 
 /// This constructor creates an empty HTTP::Parser, ready for use for either reading or writing.
 /// All this constructor does is call HTTP::Parser::Clean().

lib/sdp_media.cpp

@@ -2,6 +2,7 @@
 #include "sdp_media.h"
 #include <algorithm>
 #include <cstdarg>
+#include <sstream>
 
 namespace SDP{
 

lib/socket.cpp

@@ -5,7 +5,6 @@
 #include "defines.h"
 #include "socket.h"
 #include "timing.h"
-#include "json.h"
 #include <cstdlib>
 #include <ifaddrs.h>
 #include <netdb.h>
@@ -15,6 +14,7 @@
 #include <sys/socket.h>
 #include <sys/stat.h>
 #include <fstream>
+#include <sys/select.h>
 
 #define BUFFER_BLOCKSIZE 4096 // set buffer blocksize to 4KiB
 
@@ -843,8 +843,8 @@ void Socket::Connection::open(std::string host, int port, bool nonblock, bool wi
     }
     DONTEVEN_MSG("SSL connect");
     int ret = 0;
-    if ((ret = mbedtls_net_connect(server_fd, host.c_str(), JSON::Value(port).asString().c_str(),
-                                   MBEDTLS_NET_PROTO_TCP)) != 0){
+    std::string portStr = uint2string(port);
+    if ((ret = mbedtls_net_connect(server_fd, host.c_str(), portStr.c_str(), MBEDTLS_NET_PROTO_TCP)) != 0){
       char estr[200];
       mbedtls_strerror(ret, estr, 200);
       lastErr = estr;

lib/socket.h

@@ -5,9 +5,7 @@
 #pragma once
 #include <arpa/inet.h>
 #include <deque>
-#include <errno.h>
 #include <fcntl.h>
-#include <sstream>
 #include <stdio.h>
 #include <string.h>
 #include <string>

lib/stun.cpp

@@ -1,7 +1,8 @@
 #include "checksum.h" // for crc32
 #include "defines.h"
 #include "stun.h"
-#include "socket.h"
+#include <sstream>
+#include <socket.h>
 
 /* --------------------------------------- */
 

lib/timing.cpp

@@ -7,7 +7,6 @@
 #include <sys/time.h> //for gettimeofday
 #include <sys/stat.h>
 #include <time.h>     //for time and nanosleep
-#include <sstream>
 #include <stdlib.h>
 
 // emulate clock_gettime() for OSX compatibility

lib/util.cpp

@@ -12,6 +12,7 @@
 #include <errno.h> // errno, ENOENT, EEXIST
 #include <iomanip>
 #include <iostream>
+#include <sstream>
 #include <stdio.h>
 #include <sys/stat.h> // stat
 #if defined(_WIN32)