Fix a bug whereby it would not start to play the stream -- visible when used as system sound. Improperly initialised first_packet_timestamp

configure.ac

@@ -2,7 +2,7 @@
 # Process this file with autoconf to produce a configure script.
 
 AC_PREREQ([2.50])
-AC_INIT([shairport-sync], [2.5.0.7], [mikebrady@eircom.net])
+AC_INIT([shairport-sync], [2.5.0.8], [mikebrady@eircom.net])
 AM_INIT_AUTOMAKE
 AC_CONFIG_SRCDIR([shairport.c])
 AC_CONFIG_HEADERS([config.h])

player.c

@@ -911,6 +911,7 @@ static void *player_thread_func(void *arg) {
   silence = malloc(OUTFRAME_BYTES(frame_size));
   memset(silence, 0, OUTFRAME_BYTES(frame_size));
   late_packet_message_sent = 0;
+  first_packet_timestamp = 0;
   missing_packets = late_packets = too_late_packets = resend_requests = 0;
   flush_rtp_timestamp = 0; // it seems this number has a special significance -- it seems to be used
                            // as a null operand, so we'll use it like that too

rtsp.c

@@ -722,7 +722,7 @@ static void handle_setup(rtsp_conn_info *conn, rtsp_message *req, rtsp_message *
   
   if (config.latency==-1) {
     // this means that no static latency was set, so we'll allow it to be set dynamically
-    config.latency=88200; // to be sure, to be sure
+    config.latency=88198; // to be sure, to be sure -- make it slighty different from the default to ensure we get a debug message when set to 88200
     config.use_negotiated_latencies = 1;
   }
   char *hdr = msg_get_header(req, "Transport");