Changeset 5799 for pjproject/trunk/pjmedia/src/pjmedia/stream.c

Timestamp:

May 30, 2018 7:41:23 AM (5 years ago)

Author:

nanang

Message:

Re #2089:

• Set default max Opus RX frame length, i.e: param.info.max_rx_frame_size, to 1275. This is to avoid issue of truncated frame in jitter buffer when incoming frame length gets increased (after remote Opus encoder changes Opus params on the fly, e.g: ptime, VBR/CBR, sampling rate). Thanks Marcus Froeschl for the report, the investigation, and the solution.
• Fixed pjmedia_codec_opus_set_default_param() to initiate codec param with current default settings.
• Fixed some compile warnings.

File:

• pjproject/trunk/pjmedia/src/pjmedia/stream.c (modified) (1 diff)

pjproject/trunk/pjmedia/src/pjmedia/stream.c

@@ -1886 +1886 @@
             stream->rtp_rx_ts_len_per_frame= stream->rtp_rx_ts_len_per_frame *
                                              dec_ptime / stream->dec_ptime;
-            stream->dec_ptime = dec_ptime;
+            stream->dec_ptime = (pj_uint16_t)dec_ptime;
             pjmedia_jbuf_set_ptime(stream->jb, stream->dec_ptime);
         }