Mesa (master): vc4: Remove dead code in vc4_dump_surface_msaa()

Eric Anholt anholt at kemper.freedesktop.org
Mon May 22 16:53:42 UTC 2017


Module: Mesa
Branch: master
Commit: 499f45163a35f1dbaa0eea30e41c889e4aa4cb3a
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=499f45163a35f1dbaa0eea30e41c889e4aa4cb3a

Author: Rhys Kidd <rhyskidd at gmail.com>
Date:   Sun May 21 23:42:47 2017 -0400

vc4: Remove dead code in vc4_dump_surface_msaa()

Coverity caught the use of dead code copy-paste for
found_colors[] and num_found_colors.

CID: 1341850
Signed-off-by: Rhys Kidd <rhyskidd at gmail.com>
Reviewed-by: Eric Anholt <eric at anholt.net>

---

 src/gallium/drivers/vc4/vc4_resource.c | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/src/gallium/drivers/vc4/vc4_resource.c b/src/gallium/drivers/vc4/vc4_resource.c
index 8a5398d9f7..db7faf7ce3 100644
--- a/src/gallium/drivers/vc4/vc4_resource.c
+++ b/src/gallium/drivers/vc4/vc4_resource.c
@@ -881,8 +881,6 @@ vc4_dump_surface_msaa(struct pipe_surface *psurf)
         uint32_t char_w = 140, char_h = 60;
         uint32_t char_w_per_tile = char_w / tiles_w - 1;
         uint32_t char_h_per_tile = char_h / tiles_h - 1;
-        uint32_t found_colors[10];
-        uint32_t num_found_colors = 0;
 
         fprintf(stderr, "Surface: %dx%d (%dx MSAA)\n",
                 psurf->width, psurf->height, psurf->texture->nr_samples);
@@ -920,10 +918,6 @@ vc4_dump_surface_msaa(struct pipe_surface *psurf)
                         fprintf(stderr, "-");
                 fprintf(stderr, "\n");
         }
-
-        for (int i = 0; i < num_found_colors; i++) {
-                fprintf(stderr, "color %d: 0x%08x\n", i, found_colors[i]);
-        }
 }
 
 /** Debug routine to dump the contents of an 8888 surface to the console */




More information about the mesa-commit mailing list