Make av_parse_color() return AVERROR(EINVAL) rather than -1.
authorStefano Sabatini <stefano.sabatini-lala@poste.it>
Thu, 13 May 2010 18:26:03 +0000 (18:26 +0000)
committerStefano Sabatini <stefano.sabatini-lala@poste.it>
Thu, 13 May 2010 18:26:03 +0000 (18:26 +0000)
Originally committed as revision 23120 to svn://svn.ffmpeg.org/ffmpeg/trunk

libavfilter/parseutils.c

index 222ac73..9a7eef0 100644 (file)
@@ -231,7 +231,7 @@ int av_parse_color(uint8_t *rgba_color, const char *color_string, void *log_ctx)
 
         if (*tail || (len != 8 && len != 10)) {
             av_log(log_ctx, AV_LOG_ERROR, "Invalid 0xRRGGBB[AA] color string: '%s'\n", color_string);
-            return -1;
+            return AVERROR(EINVAL);
         }
         if (len == 10) {
             rgba_color[3] = rgba;
@@ -248,7 +248,7 @@ int av_parse_color(uint8_t *rgba_color, const char *color_string, void *log_ctx)
                                           color_table_compare);
         if (!entry) {
             av_log(log_ctx, AV_LOG_ERROR, "Cannot find color '%s'\n", color_string);
-            return -1;
+            return AVERROR(EINVAL);
         }
         memcpy(rgba_color, entry->rgba_color, 4);
     }