shithub: libvpx

Info  •  Files  •  Log  •  Branches

Fix formatting in internal stats for vp8 and vp9

This corrects a formatting error introduced in:
I1e9d548ce445d29002f0c59ebfd3957a6f15e702
where spaces were used as delimiters instead of tabs.

The corresponding fix for vp10 is in
Ica3d625d6672b3c47e0e208b45eede29b9004030.

Change-Id: Ibc4eb8fd82e6b926ba259a679dc98557cadba9b1

--- a/vp8/encoder/onyx_if.c

+++ b/vp8/encoder/onyx_if.c

@@ -2087,11 +2087,11 @@

           fprintf(f,

                   "Bitrate\tAVGPsnr\tGLBPsnr\tAVPsnrP\t"

-                  "GLPsnrP\tVPXSSIM\t  Time(us)  Rc-Err "

+                  "GLPsnrP\tVPXSSIM\tTime(us)\tRc-Err\t"

                   "Abs Err\n");

           fprintf(f,

                   "%7.3f\t%7.3f\t%7.3f\t%7.3f\t%7.3f\t"

-                  "%7.3f\t%8.0f %7.2f %7.2f\n",

+                  "%7.3f\t%8.0f\t%7.2f\t%7.2f\n",

                   dr, cpi->total / cpi->count, total_psnr,

                   cpi->totalp / cpi->count, total_psnr2, total_ssim,

                   total_encode_time, rate_err, fabs(rate_err));

--- a/vp9/encoder/vp9_encoder.c

+++ b/vp9/encoder/vp9_encoder.c

@@ -1984,9 +1984,11 @@

           SNPRINT2(results, "\t%7.3f", consistency);

           SNPRINT2(results, "\t%7.3f", cpi->worst_consistency);

         }

-        fprintf(f, "%s\t    Time  Rc-Err Abs Err\n", headings);

-        fprintf(f, "%s\t%8.0f %7.2f %7.2f\n", results, total_encode_time,

-                rate_err, fabs(rate_err));

+

+        SNPRINT(headings, "\tTime\tRcErr\tAbsErr");

+        SNPRINT2(results, "\t%7.3f", total_encode_time);

+        SNPRINT2(results, "\t%7.3f", rate_err);

+        SNPRINT2(results, "\t%7.3f", fabs(rate_err));

       }

       fclose(f);