mod_skinny: move hardwired ptime to a define so it's in one place

diff --git a/src/mod/endpoints/mod_skinny/skinny_protocol.h b/src/mod/endpoints/mod_skinny/skinny_protocol.h
index 8a5178302e55a7770673ca1a06da1ca5f157f3f7..870b4c578a27d50749b5d32af9a5ef1c7c87da94 100644 (file)
--- a/src/mod/endpoints/mod_skinny/skinny_protocol.h
+++ b/src/mod/endpoints/mod_skinny/skinny_protocol.h
@@ -39,6 +39,10 @@
 /*****************************************************************************/
 /* SKINNY TYPES */
 /*****************************************************************************/
+
+/* Hardcode ptime in one place until we make it dynamic */
+#define SKINNY_PTIME 20
+
 typedef enum {
        SKINNY_CODEC_NONE = 0,
        SKINNY_CODEC_NONSTANDARD = 1,

diff --git a/src/mod/endpoints/mod_skinny/skinny_server.c b/src/mod/endpoints/mod_skinny/skinny_server.c
index aaf7586dafff1ba419df19387c1be8a84d735ed6..b9ec3de21b9196f53df34dbb419959373bb8f313 100644 (file)
--- a/src/mod/endpoints/mod_skinny/skinny_server.c
+++ b/src/mod/endpoints/mod_skinny/skinny_server.c
@@ -788,7 +788,7 @@ switch_status_t skinny_session_start_media(switch_core_session_t *session, liste
                send_open_receive_channel(listener,
                                tech_pvt->call_id, /* uint32_t conference_id, */
                                tech_pvt->call_id, /* uint32_t pass_thru_party_id, */
-                               20, /* uint32_t ms_per_packet, */
+                               SKINNY_PTIME, /* uint32_t ms_per_packet, */
                                SKINNY_CODEC_ULAW_64K, /* uint32_t payload_capacity, */
                                0, /* uint32_t echo_cancel_type, */
                                0, /* uint32_t g723_bitrate, */
@@ -854,7 +854,7 @@ switch_status_t skinny_session_unhold_line(switch_core_session_t *session, liste
        send_open_receive_channel(listener,
                tech_pvt->call_id, /* uint32_t conference_id, */
                tech_pvt->call_id, /* uint32_t pass_thru_party_id, */
-               20, /* uint32_t ms_per_packet, */
+               SKINNY_PTIME, /* uint32_t ms_per_packet, */
                SKINNY_CODEC_ULAW_64K, /* uint32_t payload_capacity, */
                0, /* uint32_t echo_cancel_type, */
                0, /* uint32_t g723_bitrate, */
@@ -1830,7 +1830,7 @@ switch_status_t skinny_handle_open_receive_channel_ack_message(listener_t *liste
 
                /* Codec */
                tech_pvt->iananame = "PCMU"; /* TODO */
-               tech_pvt->codec_ms = 20; /* TODO */
+               tech_pvt->codec_ms = SKINNY_PTIME; /* TODO */
                tech_pvt->rm_rate = 8000; /* TODO */
                tech_pvt->rm_fmtp = NULL; /* TODO */
                tech_pvt->agreed_pt = (switch_payload_t) 0; /* TODO */
@@ -1879,7 +1879,7 @@ switch_status_t skinny_handle_open_receive_channel_ack_message(listener_t *liste
                                tech_pvt->party_id, /* uint32_t pass_thru_party_id, */
                                addr.s_addr, /* uint32_t remote_ip, */
                                tech_pvt->local_sdp_audio_port, /* uint32_t remote_port, */
-                               20, /* uint32_t ms_per_packet, */
+                               SKINNY_PTIME, /* uint32_t ms_per_packet, */
                                SKINNY_CODEC_ULAW_64K, /* uint32_t payload_capacity, */
                                184, /* uint32_t precedence, */
                                0, /* uint32_t silence_suppression, */