vpxdec.sh: fix vp9_webm_less_than_50_frames w/valgrind
authorJames Zern <jzern@google.com>
Wed, 8 Apr 2015 18:45:04 +0000 (11:45 -0700)
committerJames Zern <jzern@google.com>
Wed, 8 Apr 2015 18:45:04 +0000 (11:45 -0700)
add a check for the status line to awk and better report failure given
the program output will be lost in this case

Change-Id: I1348a80108c81099d609f2e2227dd2c31bd8cd54

test/vpxdec.sh

index 2f23ca7..de51c80 100755 (executable)
@@ -96,9 +96,14 @@ vpxdec_vp9_webm_less_than_50_frames() {
   if [ "$(vpxdec_can_decode_vp9)" = "yes" ] && \
      [ "$(webm_io_available)" = "yes" ]; then
     local readonly decoder="$(vpx_tool_path vpxdec)"
-    eval "${VPX_TEST_PREFIX}" "${decoder}" "${VP9_LT_50_FRAMES_WEBM_FILE}" \
-      --summary --noblit 2>&1 \
-      | awk '{ exit $1 != 10; }'
+    local readonly expected=10
+    local readonly num_frames=$(${VPX_TEST_PREFIX} "${decoder}" \
+      "${VP9_LT_50_FRAMES_WEBM_FILE}" --summary --noblit 2>&1 \
+      | awk '/^[0-9]+ decoded frames/ { print $1 }')
+    if [ "$num_frames" -ne "$expected" ]; then
+      elog "Output frames ($num_frames) != expected ($expected)"
+      return 1
+    fi
   fi
 }