]> git.sesse.net Git - ffmpeg/commitdiff
avformat/rtsp: Fix build failure when RTP demuxers are disabled
authorAndreas Rheinhardt <andreas.rheinhardt@gmail.com>
Fri, 29 Jan 2021 17:53:43 +0000 (18:53 +0100)
committerAndreas Rheinhardt <andreas.rheinhardt@gmail.com>
Wed, 3 Feb 2021 20:36:47 +0000 (21:36 +0100)
rtsp.c uses a check of the form "if (CONFIG_RTSP_DEMUXER && ...) {}"
with the intent to make the code compilable even though the part guarded
by this check contains calls to functions that don't exist when the RTSP
demuxer is disabled. Yet even then compilers still need a declaration of
all the functions in the dead code block and error out if not (due to
our usage of -Werror=implicit-function-declaration) and no such
declaration exists for a static function in rtsp.c. Simply adding a
declaration leads to a "used but never defined" warning, therefore this
commit resorts to an #if.

Reviewed-by: Paul B Mahol <onemda@gmail.com>
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
libavformat/rtsp.c

index 1b24496f3c699b844f1f80c538d74b60d4d636df..9a2933346eddc860b2f9ae9a4e8ff2331d8146a7 100644 (file)
@@ -1941,12 +1941,15 @@ redirect:
         break;
     }
 
-    if (CONFIG_RTSP_DEMUXER && s->iformat) {
+#if CONFIG_RTSP_DEMUXER
+    if (s->iformat) {
         if (rt->server_type == RTSP_SERVER_SATIP)
             err = init_satip_stream(s);
         else
             err = ff_rtsp_setup_input_streams(s, reply);
-    } else if (CONFIG_RTSP_MUXER)
+    } else
+#endif
+           if (CONFIG_RTSP_MUXER)
         err = ff_rtsp_setup_output_streams(s, host);
     else
         av_assert0(0);