[FFmpeg-devel] [PATCH 6/7] Make av_parse_video_frame_size() and av_parse_video_frame_rate() return AVERROR(EINVAL) rather than -1 in case of unsuccess.

Stefano Sabatini stefano.sabatini-lala
Sun Jul 25 13:51:53 CEST 2010


---
 libavcore/parseutils.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/libavcore/parseutils.c b/libavcore/parseutils.c
index bc592a1..9e3ad81 100644
--- a/libavcore/parseutils.c
+++ b/libavcore/parseutils.c
@@ -107,7 +107,7 @@ int av_parse_video_frame_size(int *width_ptr, int *height_ptr, const char *str)
         frame_height = strtol(p, &p, 10);
     }
     if (frame_width <= 0 || frame_height <= 0)
-        return -1;
+        return AVERROR(EINVAL);
     *width_ptr = frame_width;
     *height_ptr = frame_height;
     return 0;
@@ -145,7 +145,7 @@ int av_parse_video_frame_rate(AVRational *frame_rate, const char *arg)
         frame_rate->num = time_base.num;
     }
     if (!frame_rate->num || !frame_rate->den)
-        return -1;
+        return AVERROR(EINVAL);
     else
         return 0;
 }
-- 
1.7.0.4




More information about the ffmpeg-devel mailing list