[FFmpeg-devel] [PATCH] Do not return -1 from void function

Carl Eugen Hoyos cehoyos
Thu May 8 11:15:30 CEST 2008


Hi!

libschroedinger_handle_first_access_unit is void and should not return -1.

Patch attached (fixes icc warnings).

Carl Eugen
-------------- next part --------------
Index: libavcodec/libschroedingerdec.c
===================================================================
--- libavcodec/libschroedingerdec.c	(revision 13076)
+++ libavcodec/libschroedingerdec.c	(working copy)
@@ -126,7 +126,7 @@
         av_log(avccontext, AV_LOG_ERROR, "invalid dimensions (%dx%d)\n",
                p_schro_params->format->width, p_schro_params->format->height);
         avccontext->height = avccontext->width = 0;
-        return -1;
+        return;
     }
     avccontext->height  = p_schro_params->format->height;
     avccontext->width   = p_schro_params->format->width;
@@ -138,7 +138,7 @@
         av_log (avccontext, AV_LOG_ERROR,
                 "This codec currently only supports planar YUV 4:2:0, 4:2:2 "
                 "and 4:4:4 formats.\n");
-        return -1;
+        return;
     }
 
     avccontext->time_base.den = p_schro_params->format->frame_rate_numerator;



More information about the ffmpeg-devel mailing list