]> git.ipfire.org Git - thirdparty/shairport-sync.git/commitdiff
clang format
authorMike Brady <4265913+mikebrady@users.noreply.github.com>
Sat, 1 May 2021 13:09:38 +0000 (14:09 +0100)
committerMike Brady <4265913+mikebrady@users.noreply.github.com>
Sat, 1 May 2021 13:09:38 +0000 (14:09 +0100)
audio_alsa.c
player.c
player.h
ptp-utilities.c
rtp.c
rtp.h
rtsp.c
shairport.c

index 19127d1111c6c6629058cebe43f5afa7d30765ec..d5757befd550e0078efff3a19508304b995c270f 100644 (file)
@@ -1457,30 +1457,29 @@ int precision_delay_and_status(snd_pcm_state_t *state, snd_pcm_sframes_t *delay,
 
     snd_pcm_status_get_htstamp(alsa_snd_pcm_status, &update_timestamp);
 
-/*
-// must be 1.1 or later to use snd_pcm_status_get_driver_htstamp
-#if SND_LIB_MINOR != 0
-    snd_htimestamp_t driver_htstamp;
-    snd_pcm_status_get_driver_htstamp(alsa_snd_pcm_status, &driver_htstamp);
-    uint64_t driver_htstamp_ns = driver_htstamp.tv_sec;
-    driver_htstamp_ns = driver_htstamp_ns * 1000000000;
-    driver_htstamp_ns = driver_htstamp_ns + driver_htstamp.tv_nsec;
-    debug(1,"driver_htstamp: %f.", driver_htstamp_ns * 0.000000001);
-#endif
-*/
+    /*
+    // must be 1.1 or later to use snd_pcm_status_get_driver_htstamp
+    #if SND_LIB_MINOR != 0
+        snd_htimestamp_t driver_htstamp;
+        snd_pcm_status_get_driver_htstamp(alsa_snd_pcm_status, &driver_htstamp);
+        uint64_t driver_htstamp_ns = driver_htstamp.tv_sec;
+        driver_htstamp_ns = driver_htstamp_ns * 1000000000;
+        driver_htstamp_ns = driver_htstamp_ns + driver_htstamp.tv_nsec;
+        debug(1,"driver_htstamp: %f.", driver_htstamp_ns * 0.000000001);
+    #endif
+    */
 
     *state = snd_pcm_status_get_state(alsa_snd_pcm_status);
 
     if ((*state == SND_PCM_STATE_RUNNING) || (*state == SND_PCM_STATE_DRAINING)) {
 
- //     uint64_t update_timestamp_ns =
- //         update_timestamp.tv_sec * (uint64_t)1000000000 + update_timestamp.tv_nsec;
     //     uint64_t update_timestamp_ns =
     //         update_timestamp.tv_sec * (uint64_t)1000000000 + update_timestamp.tv_nsec;
 
       uint64_t update_timestamp_ns = update_timestamp.tv_sec;
       update_timestamp_ns = update_timestamp_ns * 1000000000;
       update_timestamp_ns = update_timestamp_ns + update_timestamp.tv_nsec;
 
-
       // if the update_timestamp is zero, we take this to mean that the device doesn't report
       // interrupt timings. (It could be that it's not a real hardware device.)
       // so we switch to getting the delay the regular way
@@ -1531,7 +1530,6 @@ int precision_delay_and_status(snd_pcm_state_t *state, snd_pcm_sframes_t *delay,
         time_now_ns = time_now_ns * 1000000000;
         time_now_ns = time_now_ns + tn.tv_nsec;
 
-
         // see if it's stalled
 
         if ((stall_monitor_start_time != 0) && (stall_monitor_frame_count == delay_temp)) {
@@ -1561,18 +1559,22 @@ int precision_delay_and_status(snd_pcm_state_t *state, snd_pcm_sframes_t *delay,
         if (ret == 0) {
           uint64_t delta = time_now_ns - update_timestamp_ns;
 
-//          uint64_t frames_played_since_last_interrupt =
-//              ((uint64_t)config.output_rate * delta) / 1000000000;
-
-            uint64_t frames_played_since_last_interrupt = config.output_rate;
-            frames_played_since_last_interrupt = frames_played_since_last_interrupt * delta;
-            frames_played_since_last_interrupt = frames_played_since_last_interrupt / 1000000000;
+          //          uint64_t frames_played_since_last_interrupt =
+          //              ((uint64_t)config.output_rate * delta) / 1000000000;
 
+          uint64_t frames_played_since_last_interrupt = config.output_rate;
+          frames_played_since_last_interrupt = frames_played_since_last_interrupt * delta;
+          frames_played_since_last_interrupt = frames_played_since_last_interrupt / 1000000000;
 
           snd_pcm_sframes_t frames_played_since_last_interrupt_sized =
               frames_played_since_last_interrupt;
-          if ((frames_played_since_last_interrupt_sized < 0) || ((uint64_t)frames_played_since_last_interrupt_sized != frames_played_since_last_interrupt))
-            debug(1,"overflow resizing frames_played_since_last_interrupt % " PRIx64 " to frames_played_since_last_interrupt %lx.", frames_played_since_last_interrupt, frames_played_since_last_interrupt_sized);
+          if ((frames_played_since_last_interrupt_sized < 0) ||
+              ((uint64_t)frames_played_since_last_interrupt_sized !=
+               frames_played_since_last_interrupt))
+            debug(1,
+                  "overflow resizing frames_played_since_last_interrupt % " PRIx64
+                  " to frames_played_since_last_interrupt %lx.",
+                  frames_played_since_last_interrupt, frames_played_since_last_interrupt_sized);
           delay_temp = delay_temp - frames_played_since_last_interrupt_sized;
         }
         *delay = delay_temp;
index 66637292caa8dc68b236c50a83a0626878d4bbc1..6b67ee1331e11b01613361875e24f9e2ae7062d7 100644 (file)
--- a/player.c
+++ b/player.c
@@ -435,93 +435,93 @@ void player_put_packet(int original_format, seq_t seqno, uint32_t actual_timesta
   conn->packet_count_since_flush++;
   conn->time_of_last_audio_packet = time_now;
   if (conn->connection_state_to_output) { // if we are supposed to be processing these packets
-      abuf_t *abuf = 0;
-      if (!conn->ab_synced) {
-        // if this is the first packet...
-        debug(3, "syncing to seqno %u.", seqno);
-        conn->ab_write = seqno;
-        conn->ab_read = seqno;
-        conn->ab_synced = 1;
-      } else if (original_format == 0) {
-        // if the packet is coming in original format, the sequence number is important
-        // otherwise, ignore is by setting it equal to the expected sequence number in ab_write
-        seqno = conn->ab_write;
-      }
-      if (conn->ab_write ==
-          seqno) { // if this is the expected packet (which could be the first packet...)
-        if (conn->input_frame_rate_starting_point_is_valid == 0) {
-          if ((conn->packet_count_since_flush >= 500) && (conn->packet_count_since_flush <= 510)) {
-            conn->frames_inward_measurement_start_time = time_now;
-            conn->frames_inward_frames_received_at_measurement_start_time = actual_timestamp;
-            conn->input_frame_rate_starting_point_is_valid = 1; // valid now
-          }
-        }
-        conn->frames_inward_measurement_time = time_now;
-        conn->frames_inward_frames_received_at_measurement_time = actual_timestamp;
-        abuf = conn->audio_buffer + BUFIDX(seqno);
-        conn->ab_write = seqno + 1; // move the write pointer to the next free space
-      } else if (seq_order(conn->ab_write, seqno, conn->ab_read)) { // newer than expected
-        int32_t gap = seq_diff(conn->ab_write, seqno, conn->ab_read);
-        if (gap <= 0)
-          debug(1, "Unexpected gap size: %d.", gap);
-        int i;
-        for (i = 0; i < gap; i++) {
-          abuf = conn->audio_buffer + BUFIDX(seq_sum(conn->ab_write, i));
-          abuf->ready = 0; // to be sure, to be sure
-          abuf->resend_request_number = 0;
-          abuf->initialisation_time =
-              time_now;          // this represents when the packet was noticed to be missing
-          abuf->status = 1 << 0; // signifying missing
-          abuf->resend_time = 0;
-          abuf->given_timestamp = 0;
-          abuf->sequence_number = 0;
+    abuf_t *abuf = 0;
+    if (!conn->ab_synced) {
+      // if this is the first packet...
+      debug(3, "syncing to seqno %u.", seqno);
+      conn->ab_write = seqno;
+      conn->ab_read = seqno;
+      conn->ab_synced = 1;
+    } else if (original_format == 0) {
+      // if the packet is coming in original format, the sequence number is important
+      // otherwise, ignore is by setting it equal to the expected sequence number in ab_write
+      seqno = conn->ab_write;
+    }
+    if (conn->ab_write ==
+        seqno) { // if this is the expected packet (which could be the first packet...)
+      if (conn->input_frame_rate_starting_point_is_valid == 0) {
+        if ((conn->packet_count_since_flush >= 500) && (conn->packet_count_since_flush <= 510)) {
+          conn->frames_inward_measurement_start_time = time_now;
+          conn->frames_inward_frames_received_at_measurement_start_time = actual_timestamp;
+          conn->input_frame_rate_starting_point_is_valid = 1; // valid now
         }
-        // debug(1,"N %d s %u.",seq_diff(ab_write,PREDECESSOR(seqno))+1,ab_write);
-        abuf = conn->audio_buffer + BUFIDX(seqno);
-        //        rtp_request_resend(ab_write, gap);
-        //        resend_requests++;
-        conn->ab_write = seqno + 1;
-      } else if (seq_order(conn->ab_read, seqno,
-                           conn->ab_read)) { // older than expected but not too late
-        conn->late_packets++;
-        abuf = conn->audio_buffer + BUFIDX(seqno);
-      } else { // too late.
-        conn->too_late_packets++;
       }
-
-      if (abuf) {
-        int datalen = conn->max_frames_per_packet;
-        abuf->initialisation_time = time_now;
+      conn->frames_inward_measurement_time = time_now;
+      conn->frames_inward_frames_received_at_measurement_time = actual_timestamp;
+      abuf = conn->audio_buffer + BUFIDX(seqno);
+      conn->ab_write = seqno + 1; // move the write pointer to the next free space
+    } else if (seq_order(conn->ab_write, seqno, conn->ab_read)) { // newer than expected
+      int32_t gap = seq_diff(conn->ab_write, seqno, conn->ab_read);
+      if (gap <= 0)
+        debug(1, "Unexpected gap size: %d.", gap);
+      int i;
+      for (i = 0; i < gap; i++) {
+        abuf = conn->audio_buffer + BUFIDX(seq_sum(conn->ab_write, i));
+        abuf->ready = 0; // to be sure, to be sure
+        abuf->resend_request_number = 0;
+        abuf->initialisation_time =
+            time_now;          // this represents when the packet was noticed to be missing
+        abuf->status = 1 << 0; // signifying missing
         abuf->resend_time = 0;
-        if ((original_format != 0) &&
-            (audio_packet_decode(abuf->data, &datalen, data, len, conn) == 0)) {
-          abuf->ready = 1;
-          abuf->status = 0; // signifying that it was received
-          abuf->length = datalen;
-          abuf->given_timestamp = actual_timestamp;
-          abuf->sequence_number = seqno;
-        } else if (original_format == 0) {
-          memcpy(abuf->data, data, len * conn->input_bytes_per_frame);
-          abuf->ready = 1;
-          abuf->status = 0; // signifying that it was received
-          abuf->length = len;
-          abuf->given_timestamp = actual_timestamp;
-          abuf->sequence_number = seqno;
-        } else {
-          debug(1, "Bad audio packet detected and discarded.");
-          abuf->ready = 0;
-          abuf->status = 1 << 1; // bad packet, discarded
-          abuf->resend_request_number = 0;
-          abuf->given_timestamp = 0;
-          abuf->sequence_number = 0;
-        }
+        abuf->given_timestamp = 0;
+        abuf->sequence_number = 0;
       }
+      // debug(1,"N %d s %u.",seq_diff(ab_write,PREDECESSOR(seqno))+1,ab_write);
+      abuf = conn->audio_buffer + BUFIDX(seqno);
+      //        rtp_request_resend(ab_write, gap);
+      //        resend_requests++;
+      conn->ab_write = seqno + 1;
+    } else if (seq_order(conn->ab_read, seqno,
+                         conn->ab_read)) { // older than expected but not too late
+      conn->late_packets++;
+      abuf = conn->audio_buffer + BUFIDX(seqno);
+    } else { // too late.
+      conn->too_late_packets++;
+    }
 
-      int rc = pthread_cond_signal(&conn->flowcontrol);
-      if (rc)
-        debug(1, "Error signalling flowcontrol.");
+    if (abuf) {
+      int datalen = conn->max_frames_per_packet;
+      abuf->initialisation_time = time_now;
+      abuf->resend_time = 0;
+      if ((original_format != 0) &&
+          (audio_packet_decode(abuf->data, &datalen, data, len, conn) == 0)) {
+        abuf->ready = 1;
+        abuf->status = 0; // signifying that it was received
+        abuf->length = datalen;
+        abuf->given_timestamp = actual_timestamp;
+        abuf->sequence_number = seqno;
+      } else if (original_format == 0) {
+        memcpy(abuf->data, data, len * conn->input_bytes_per_frame);
+        abuf->ready = 1;
+        abuf->status = 0; // signifying that it was received
+        abuf->length = len;
+        abuf->given_timestamp = actual_timestamp;
+        abuf->sequence_number = seqno;
+      } else {
+        debug(1, "Bad audio packet detected and discarded.");
+        abuf->ready = 0;
+        abuf->status = 1 << 1; // bad packet, discarded
+        abuf->resend_request_number = 0;
+        abuf->given_timestamp = 0;
+        abuf->sequence_number = 0;
+      }
+    }
+
+    int rc = pthread_cond_signal(&conn->flowcontrol);
+    if (rc)
+      debug(1, "Error signalling flowcontrol.");
 
-     // resend checks
+    // resend checks
     {
       uint64_t minimum_wait_time =
           (uint64_t)(config.resend_control_first_check_time * (uint64_t)1000000000);
@@ -1050,7 +1050,6 @@ static abuf_t *buffer_get_frame(rtsp_conn_info *conn) {
               // The desired latency, typically 88200 frames, will be calculated for in rtp.c,
               // and any desired backend latency offset included in it there.
 
-
               uint64_t should_be_time;
 
               frame_to_local_time(conn->first_packet_timestamp, // this will go modulo 2^32
@@ -1243,9 +1242,11 @@ static abuf_t *buffer_get_frame(rtsp_conn_info *conn) {
         // we must enable packets to be released early enough for the
         // audio buffer to be filled to the desired length
 
-        uint32_t buffer_latency_offset = (uint32_t)(config.audio_backend_buffer_desired_length * conn->input_rate);
-        frame_to_local_time(curframe->given_timestamp - buffer_latency_offset, // this will go modulo 2^32
-                                &time_to_play, conn);
+        uint32_t buffer_latency_offset =
+            (uint32_t)(config.audio_backend_buffer_desired_length * conn->input_rate);
+        frame_to_local_time(curframe->given_timestamp -
+                                buffer_latency_offset, // this will go modulo 2^32
+                            &time_to_play, conn);
 
         if (local_time_now >= time_to_play) {
           do_wait = 0;
@@ -2984,7 +2985,7 @@ void player_flush(uint32_t timestamp, rtsp_conn_info *conn) {
   debug(3, "player_flush");
   do_flush(timestamp, conn);
 #ifdef CONFIG_METADATA
-// only send a flush metadata message if the first packet has been seen -- it's a bogus message
+  // only send a flush metadata message if the first packet has been seen -- it's a bogus message
   // otherwise
   if (conn->first_packet_timestamp) {
     debug(2, "pfls");
index 849860fd556e93df74ee306d9f7ddd055e153f06..9a92c0c4ff6f2e703d592315f7ab7815192e7ec2 100644 (file)
--- a/player.h
+++ b/player.h
@@ -81,10 +81,7 @@ typedef enum {
   ast_apple_lossless,
 } audio_stream_type;
 
-typedef enum {
-  ts_ntp,
-  ts_ptp
-} timing_source_type;
+typedef enum { ts_ntp, ts_ptp } timing_source_type;
 
 typedef struct {
   int encrypted;
@@ -105,10 +102,10 @@ typedef struct file_cipher_context {
 #endif
 
 typedef struct {
-  int connection_number;     // for debug ID purposes, nothing else...
+  int connection_number;             // for debug ID purposes, nothing else...
   timing_source_type type_of_timing; // are we using NTP or PTP?
-  int resend_interval;       // this is really just for debugging
-  char *UserAgent;           // free this on teardown
+  int resend_interval;               // this is really just for debugging
+  char *UserAgent;                   // free this on teardown
   int AirPlayVersion;        // zero if not an AirPlay session. Used to help calculate latency
   uint32_t latency;          // the actual latency used for this play session
   uint32_t minimum_latency;  // set if an a=min-latency: line appears in the ANNOUNCE message; zero
@@ -117,7 +114,6 @@ typedef struct {
                              // otherwise
   int software_mute_enabled; // if we don't have a real mute that we can use
 
-
   int fd;
   int authorized;   // set if a password is required and has been supplied
   char *auth_nonce; // the session nonce, if needed
@@ -216,11 +212,11 @@ typedef struct {
 
   char client_ip_string[INET6_ADDRSTRLEN]; // the ip string pointing to the client
   uint16_t client_rtsp_port;
-  char self_ip_string[INET6_ADDRSTRLEN];   // the ip string being used by this program -- it
-  uint16_t self_rtsp_port;                 // could be one of many, so we need to know it
+  char self_ip_string[INET6_ADDRSTRLEN]; // the ip string being used by this program -- it
+  uint16_t self_rtsp_port;               // could be one of many, so we need to know it
 
-  uint32_t self_scope_id;                  // if it's an ipv6 connection, this will be its scope
-  short connection_ip_family;              // AF_INET / AF_INET6
+  uint32_t self_scope_id;     // if it's an ipv6 connection, this will be its scope
+  short connection_ip_family; // AF_INET / AF_INET6
 
   SOCKADDR rtp_client_control_socket; // a socket pointing to the control port of the client
   SOCKADDR rtp_client_timing_socket;  // a socket pointing to the timing port of the client
@@ -243,7 +239,6 @@ typedef struct {
   uint64_t anchor_time; // this is the time according to the clock
   uint32_t anchor_rtptime;
 
-
 #ifdef CONFIG_AIRPLAY_2
   pthread_t rtp_event_thread;
   pthread_t rtp_ap2_control_thread;
@@ -259,7 +254,6 @@ typedef struct {
   uint64_t last_anchor_time_of_update;
   uint64_t last_anchor_clock;
 
-
   ssize_t ap2_audio_buffer_size;
   int ap2_flush_requested;
   uint32_t ap2_flush_rtp_timestamp;
@@ -285,7 +279,7 @@ typedef struct {
   unsigned char *session_key; // needs to be free'd at the end
   uint64_t frames_packet;
   uint64_t type;
-  uint64_t networkTimeTimelineID;           // the clock ID used by the player
+  uint64_t networkTimeTimelineID; // the clock ID used by the player
 
   char *ap2_timing_peer_list_message;
 
@@ -357,8 +351,8 @@ void player_volume(double f, rtsp_conn_info *conn);
 void player_volume_without_notification(double f, rtsp_conn_info *conn);
 void player_flush(uint32_t timestamp, rtsp_conn_info *conn);
 void player_full_flush(rtsp_conn_info *conn);
-void player_put_packet(int original_format, seq_t seqno, uint32_t actual_timestamp, uint8_t *data, int len,
-                       rtsp_conn_info *conn);
+void player_put_packet(int original_format, seq_t seqno, uint32_t actual_timestamp, uint8_t *data,
+                       int len, rtsp_conn_info *conn);
 int64_t monotonic_timestamp(uint32_t timestamp,
                             rtsp_conn_info *conn); // add an epoch to the timestamp. The monotonic
 // timestamp guaranteed to start between 2^32 2^33
index 887005220b06ace3dc5692db18e67e166dc44012..01d32c58f3ee8ae52bf5a65c6c7e1d063472bdf0 100644 (file)
@@ -83,12 +83,15 @@ int ptp_get_clock_info(uint64_t *actual_clock_id, uint64_t *raw_offset) {
             *raw_offset = nqptp_data.local_to_master_time_offset;
           response = 0;
         } else {
-          debug(1,"clock not valid");
+          debug(1, "clock not valid");
           response = -2; // clock info not valid
         }
       } else {
         if (failure_message_sent == 0) {
-          warn("This version of Shairport Sync requires an NQPTP with a Shared Memory Interface Version %u, but the installed version is %u. Please install the correct version of NQPTP.", NQPTP_SHM_STRUCTURES_VERSION, nqptp_data.version);
+          warn("This version of Shairport Sync requires an NQPTP with a Shared Memory Interface "
+               "Version %u, but the installed version is %u. Please install the correct version of "
+               "NQPTP.",
+               NQPTP_SHM_STRUCTURES_VERSION, nqptp_data.version);
           failure_message_sent = 1;
         }
       }
diff --git a/rtp.c b/rtp.c
index 88628ebf4fbe5fa5798e07c5a1bcd946d2b2b416..35e415a9498a4aaf9450630d9a01d92925e0ac00 100644 (file)
--- a/rtp.c
+++ b/rtp.c
@@ -53,7 +53,6 @@
 #include <sodium.h>
 #endif
 
-
 struct Nvll {
   char *name;
   double value;
@@ -289,8 +288,8 @@ void *rtp_control_receiver(void *arg) {
                                                                 obfp += 2;
                                                               };
                                                               *obfp = 0;
-
-
+                                             
+                                             
                                                               // get raw timestamp information
                                                               // I think that a good way to understand these timestamps is that
                                                               // (1) the rtlt below is the timestamp of the frame that should be playing at the
@@ -301,19 +300,19 @@ void *rtp_control_receiver(void *arg) {
                                                               // Thus, (3) the latency can be calculated by subtracting the second from the
                                                               // first.
                                                               // There must be more to it -- there something missing.
-
+                                             
                                                               // In addition, it seems that if the value of the short represented by the second
                                                               // pair of bytes in the packet is 7
                                                               // then an extra time lag is expected to be added, presumably by
                                                               // the AirPort Express.
-
+                                             
                                                               // Best guess is that this delay is 11,025 frames.
-
+                                             
                                                               uint32_t rtlt = nctohl(&packet[4]); // raw timestamp less latency
                                                               uint32_t rt = nctohl(&packet[16]);  // raw timestamp
-
+                                             
                                                               uint32_t fl = nctohs(&packet[2]); //
-
+                                             
                                                               debug(1,"Sync Packet of %d bytes received: \"%s\", flags: %d, timestamps %u and %u,
                                                           giving a latency of %d frames.",plen,obf,fl,rt,rtlt,rt-rtlt);
                                                               //debug(1,"Monotonic timestamps are: %" PRId64 " and %" PRId64 "
@@ -390,10 +389,15 @@ void *rtp_control_receiver(void *arg) {
                      la, max_frames, conn->latency);
               } else {
 
-                // here we have the latency but it does not yet account for the audio_backend_latency_offset
-                int32_t latency_offset = (uint32_t)(config.audio_backend_latency_offset * conn->input_rate);
+                // here we have the latency but it does not yet account for the
+                // audio_backend_latency_offset
+                int32_t latency_offset =
+                    (uint32_t)(config.audio_backend_latency_offset * conn->input_rate);
                 int32_t adjusted_latency = latency_offset + (int32_t)la;
-                if ((adjusted_latency < 0) || (adjusted_latency > (int32_t)(conn->max_frames_per_packet * (BUFFER_FRAMES - config.minimum_free_buffer_headroom))))
+                if ((adjusted_latency < 0) ||
+                    (adjusted_latency >
+                     (int32_t)(conn->max_frames_per_packet *
+                               (BUFFER_FRAMES - config.minimum_free_buffer_headroom))))
                   warn("audio_backend_latency_offset out of range -- ignored.");
                 else
                   la = adjusted_latency;
@@ -403,9 +407,11 @@ void *rtp_control_receiver(void *arg) {
                   debug(1,
                         "New latency: %" PRIu32 ", sync latency: %" PRIu32
                         ", minimum latency: %" PRIu32 ", maximum "
-                        "latency: %" PRIu32 ", fixed offset: %" PRIu32 ", audio_backend_latency_offset: %f.",
+                        "latency: %" PRIu32 ", fixed offset: %" PRIu32
+                        ", audio_backend_latency_offset: %f.",
                         la, sync_rtp_timestamp - rtp_timestamp_less_latency, conn->minimum_latency,
-                        conn->maximum_latency, config.fixedLatencyOffset, config.audio_backend_latency_offset);
+                        conn->maximum_latency, config.fixedLatencyOffset,
+                        config.audio_backend_latency_offset);
                 }
               }
             }
@@ -979,11 +985,11 @@ void rtp_setup(SOCKADDR *local, SOCKADDR *remote, uint16_t cport, uint16_t tport
     conn->remote_timing_port = tport;
 
     conn->local_control_port = bind_UDP_port(conn->connection_ip_family, conn->self_ip_string,
-                                         conn->self_scope_id, &conn->control_socket);
+                                             conn->self_scope_id, &conn->control_socket);
     conn->local_timing_port = bind_UDP_port(conn->connection_ip_family, conn->self_ip_string,
-                                        conn->self_scope_id, &conn->timing_socket);
+                                            conn->self_scope_id, &conn->timing_socket);
     conn->local_audio_port = bind_UDP_port(conn->connection_ip_family, conn->self_ip_string,
-                                       conn->self_scope_id, &conn->audio_socket);
+                                           conn->self_scope_id, &conn->audio_socket);
 
     debug(3, "listening for audio, control and timing on ports %d, %d, %d.", conn->local_audio_port,
           conn->local_control_port, conn->local_timing_port);
@@ -1050,8 +1056,7 @@ int sanitised_source_rate_information(uint32_t *frames, uint64_t *time, rtsp_con
   return result;
 }
 
-void set_ntp_anchor_info(rtsp_conn_info *conn, uint32_t rtptime,
-                     uint64_t networktime) {
+void set_ntp_anchor_info(rtsp_conn_info *conn, uint32_t rtptime, uint64_t networktime) {
   conn->anchor_remote_info_is_valid = 1;
   conn->anchor_rtptime = rtptime;
   conn->anchor_time = networktime;
@@ -1071,14 +1076,14 @@ int frame_to_ntp_local_time(uint32_t timestamp, uint64_t *time, rtsp_conn_info *
   int32_t timestamp_interval = timestamp - conn->anchor_rtptime;
   int64_t timestamp_interval_time = timestamp_interval;
   timestamp_interval_time = timestamp_interval_time * time_difference;
-  timestamp_interval_time = timestamp_interval_time /
-                              frame_difference; // this is the nominal time, based on the
-                                                // fps specified between current and
-                                                // previous sync frame.
-  remote_time_of_timestamp = conn->anchor_time +
-                               timestamp_interval_time; // based on the reference timestamp time
-                                                        // plus the time interval calculated based
-                                                        // on the specified fps.
+  timestamp_interval_time =
+      timestamp_interval_time / frame_difference; // this is the nominal time, based on the
+                                                  // fps specified between current and
+                                                  // previous sync frame.
+  remote_time_of_timestamp =
+      conn->anchor_time + timestamp_interval_time; // based on the reference timestamp time
+                                                   // plus the time interval calculated based
+                                                   // on the specified fps.
   *time = remote_time_of_timestamp - local_to_remote_time_difference_now(conn);
   debug_mutex_unlock(&conn->reference_time_mutex, 0);
   return result;
@@ -1191,7 +1196,7 @@ void rtp_request_resend(seq_t first, uint32_t count, rtsp_conn_info *conn) {
 #ifdef CONFIG_AIRPLAY_2
 
 void set_ptp_anchor_info(rtsp_conn_info *conn, uint64_t clock_id, uint32_t rtptime,
-                     uint64_t networktime) {
+                         uint64_t networktime) {
   if (conn->anchor_clock != clock_id)
     debug(1, "Connection %d: Set Anchor Clock: %" PRIx64 ".", conn->connection_number, clock_id);
   conn->anchor_remote_info_is_valid = 1;
@@ -1207,7 +1212,7 @@ void reset_ptp_anchor_info(rtsp_conn_info *conn) {
 }
 
 int get_ptp_anchor_local_time_info(rtsp_conn_info *conn, uint32_t *anchorRTP,
-                               uint64_t *anchorLocalTime) {
+                                   uint64_t *anchorLocalTime) {
   int response = 0;
   uint64_t actual_clock_id, actual_offset;
 
@@ -1512,7 +1517,8 @@ void *rtp_ap2_control_receiver(void *arg) {
           // this is now only used for calculating when to ask for resends
           conn->latency = notified_latency + 11035 + added_latency;
           // debug(1,"conn->latency is %d.", conn->latency);
-          set_ptp_anchor_info(conn, clock_id, frame_1 - 11035 - added_latency, remote_packet_time_ns);
+          set_ptp_anchor_info(conn, clock_id, frame_1 - 11035 - added_latency,
+                              remote_packet_time_ns);
 
         } break;
         case 0xd6:
@@ -2359,9 +2365,7 @@ int local_time_to_frame(uint64_t time, uint32_t *frame, rtsp_conn_info *conn) {
   return local_ptp_time_to_frame(time, frame, conn);
 }
 
-void reset_anchor_info(rtsp_conn_info *conn) {
-  reset_ptp_anchor_info(conn);
-}
+void reset_anchor_info(rtsp_conn_info *conn) { reset_ptp_anchor_info(conn); }
 
 int have_timestamp_timing_information(rtsp_conn_info *conn) {
   return have_ptp_timing_information(conn);
@@ -2376,13 +2380,9 @@ int local_time_to_frame(uint64_t time, uint32_t *frame, rtsp_conn_info *conn) {
   return local_ntp_time_to_frame(time, frame, conn);
 }
 
-void reset_anchor_info(rtsp_conn_info *conn) {
-  reset_ntp_anchor_info(conn);
-}
+void reset_anchor_info(rtsp_conn_info *conn) { reset_ntp_anchor_info(conn); }
 
 int have_timestamp_timing_information(rtsp_conn_info *conn) {
   return have_ntp_timestamp_timing_information(conn);
 }
 #endif
-
-
diff --git a/rtp.h b/rtp.h
index d0a0ed012b80a3433c62b0637d023f5c44a41e02..8eff06bb437c3bac6e37233cdc97851787228d89 100644 (file)
--- a/rtp.h
+++ b/rtp.h
@@ -32,7 +32,7 @@ void *rtp_buffered_audio_processor(void *arg);
 void *rtp_ap2_timing_receiver(void *arg);
 void *rtp_ap2_general_message_timing_receiver(void *arg);
 void set_ptp_anchor_info(rtsp_conn_info *conn, uint64_t clock_id, uint32_t rtptime,
-                     uint64_t networktime);
+                         uint64_t networktime);
 #endif
 
 #endif // _RTP_H
diff --git a/rtsp.c b/rtsp.c
index 8236765bd031e8a964c82362f9a57d20d01f1a87..41ddff2265ecbef4a2b668f18a37a1eded1d79ed 100644 (file)
--- a/rtsp.c
+++ b/rtsp.c
@@ -1665,7 +1665,7 @@ void handle_setrateanchori(rtsp_conn_info *conn, rtsp_message *req, rtsp_message
 
       int32_t added_latency = (int32_t)(config.audio_backend_latency_offset * conn->input_rate);
       set_ptp_anchor_info(conn, conn->networkTimeTimelineID, anchorRTPTime - added_latency,
-                      anchorTimeNanoseconds);
+                          anchorTimeNanoseconds);
     }
 
     item = plist_dict_get_item(messagePlist, "rate");
@@ -4150,10 +4150,10 @@ void rtsp_conversation_thread_cleanup_function(void *arg) {
   debug(3, "Connection %d: Checking play lock.", conn->connection_number);
   release_play_lock(conn);
 
-  #ifdef CONFIG_AIRPLAY_2
+#ifdef CONFIG_AIRPLAY_2
   if (conn->server_setup_ctx)
     free(conn->server_setup_ctx);
-  #endif
+#endif
 
   debug(1, "Connection %d: terminated.", conn->connection_number);
   conn->running = 0;
index bdb837d837f0792f624ae7f4edd843273add0c98..2e9dff2abe4a81121a87207bf977470173cc6ae7 100644 (file)
@@ -1363,14 +1363,14 @@ void exit_function() {
       #endif
       */
 #ifdef CONFIG_DBUS_INTERFACE
-    debug(2, "Stopping D-Bus service");
-    stop_dbus_service();
+      debug(2, "Stopping D-Bus service");
+      stop_dbus_service();
 #endif
-    if (g_main_loop) {
-      debug(2, "Stopping D-Bus Loop Thread");
-      g_main_loop_quit(g_main_loop);
-      pthread_join(dbus_thread, NULL);
-    }
+      if (g_main_loop) {
+        debug(2, "Stopping D-Bus Loop Thread");
+        g_main_loop_quit(g_main_loop);
+        pthread_join(dbus_thread, NULL);
+      }
 #endif
 
 #ifdef CONFIG_DACP_CLIENT
@@ -1384,11 +1384,11 @@ void exit_function() {
 #endif
 
 #ifdef CONFIG_METADATA
-    debug(2, "Stopping metadata");
-    metadata_stop(); // close down the metadata pipe
+      debug(2, "Stopping metadata");
+      metadata_stop(); // close down the metadata pipe
 #endif
-    debug(2, "Deinitialising the audio backend.");
-    activity_monitor_stop(0);
+      debug(2, "Deinitialising the audio backend.");
+      activity_monitor_stop(0);
 
       activity_monitor_stop(0);
 
@@ -1398,9 +1398,9 @@ void exit_function() {
       }
 
 #ifdef CONFIG_SOXR
-    // be careful -- not sure if the thread can be cancelled cleanly, so wait for it to shut down
-    debug(2, "Waiting for SoXr timecheck to terminate...");
-    pthread_join(soxr_time_check_thread, NULL);
+      // be careful -- not sure if the thread can be cancelled cleanly, so wait for it to shut down
+      debug(2, "Waiting for SoXr timecheck to terminate...");
+      pthread_join(soxr_time_check_thread, NULL);
 #endif
 
       if (conns)