Changeset 6138 for pjproject/trunk/pjmedia/src/pjmedia-videodev/sdl_dev.c

Timestamp:

Jan 28, 2020 2:42:42 AM (5 years ago)

Author:

nanang

Message:

Misc (re #2210): Get rid of annoying persistent log lines due to video conference connection between active capturer and stopped/hidden renderer (thanks to Dmytrii Gonchar for the feedback).

File:

• pjproject/trunk/pjmedia/src/pjmedia-videodev/sdl_dev.c (modified) (1 diff)

pjproject/trunk/pjmedia/src/pjmedia-videodev/sdl_dev.c

@@ -926 +926 @@
     stream->last_ts.u64 = frame->timestamp.u64;
 
+    /* Video conference just trying to send heart beat for updating timestamp
+     * or keep-alive, this port doesn't need any, just ignore.
+     */
+    if (frame->size==0 || frame->buf==NULL)
+        return PJ_SUCCESS;
+
+    if (frame->size < stream->vafp.framebytes)
+        return PJ_ETOOSMALL;
+
     if (!stream->is_running)
         return PJ_EINVALIDOP;
-
-    if (frame->size==0 || frame->buf==NULL ||
-        frame->size < stream->vafp.framebytes)
-        return PJ_SUCCESS;
 
     stream->frame = frame;