[FFmpeg-devel] [PATCH 3/4] avutil/eval: check av_expr_parse_and_eval error code in the test build

[Ganesh Ajjanagadde]

This returns the error code from main in the test, in this case ENOMEM.
This should not matter, and will ensure that no warnings are triggered
when av_warn_unused_result is added to avutil/eval.h.

Signed-off-by: Ganesh Ajjanagadde <gajjanagadde at gmail.com>
---
 libavutil/eval.c | 22 +++++++++++++---------
 1 file changed, 13 insertions(+), 9 deletions(-)

diff --git a/libavutil/eval.c b/libavutil/eval.c
index 7642b91..776a3ee 100644
--- a/libavutil/eval.c
+++ b/libavutil/eval.c
@@ -750,7 +750,7 @@ static const char *const const_names[] = {
 
 int main(int argc, char **argv)
 {
-    int i;
+    int i, ret;
     double d;
     const char *const *expr;
     static const char *const exprs[] = {
@@ -854,30 +854,34 @@ int main(int argc, char **argv)
 
     for (expr = exprs; *expr; expr++) {
         printf("Evaluating '%s'\n", *expr);
-        av_expr_parse_and_eval(&d, *expr,
+        if ((ret = av_expr_parse_and_eval(&d, *expr,
                                const_names, const_values,
-                               NULL, NULL, NULL, NULL, NULL, 0, NULL);
+                               NULL, NULL, NULL, NULL, NULL, 0, NULL)) < 0)
+            return ret;
         if (isnan(d))
             printf("'%s' -> nan\n\n", *expr);
         else
             printf("'%s' -> %f\n\n", *expr, d);
     }
 
-    av_expr_parse_and_eval(&d, "1+(5-2)^(3-1)+1/2+sin(PI)-max(-2.2,-3.1)",
+    if ((ret = av_expr_parse_and_eval(&d, "1+(5-2)^(3-1)+1/2+sin(PI)-max(-2.2,-3.1)",
                            const_names, const_values,
-                           NULL, NULL, NULL, NULL, NULL, 0, NULL);
+                           NULL, NULL, NULL, NULL, NULL, 0, NULL)) < 0)
+        return ret;
     printf("%f == 12.7\n", d);
-    av_expr_parse_and_eval(&d, "80G/80Gi",
+    if ((ret = av_expr_parse_and_eval(&d, "80G/80Gi",
                            const_names, const_values,
-                           NULL, NULL, NULL, NULL, NULL, 0, NULL);
+                           NULL, NULL, NULL, NULL, NULL, 0, NULL)) < 0)
+        return ret;
     printf("%f == 0.931322575\n", d);
 
     if (argc > 1 && !strcmp(argv[1], "-t")) {
         for (i = 0; i < 1050; i++) {
             START_TIMER;
-            av_expr_parse_and_eval(&d, "1+(5-2)^(3-1)+1/2+sin(PI)-max(-2.2,-3.1)",
+            if ((ret = av_expr_parse_and_eval(&d, "1+(5-2)^(3-1)+1/2+sin(PI)-max(-2.2,-3.1)",
                                    const_names, const_values,
-                                   NULL, NULL, NULL, NULL, NULL, 0, NULL);
+                                   NULL, NULL, NULL, NULL, NULL, 0, NULL)) < 0)
+                return ret;
             STOP_TIMER("av_expr_parse_and_eval");
         }
     }
-- 
2.6.2