Mesa (master): intel/gen_decoder: Fix unused-but-set-variable warning

GitLab Mirror gitlab-mirror at kemper.freedesktop.org
Thu Nov 7 00:36:50 UTC 2019


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

Author: Kai Wasserbäch <kai at dev.carbon-project.org>
Date:   Sat Sep 28 18:58:57 2019 +0200

intel/gen_decoder: Fix unused-but-set-variable warning

This commit fixes the following warning:
../src/intel/common/gen_decoder.c: In function ‘gen_spec_load_from_path’:
../src/intel/common/gen_decoder.c:741:11: warning: variable ‘len’ set but not used [-Wunused-but-set-variable]
  741 |    size_t len, filename_len = strlen(path) + 20;
      |           ^~~

Signed-off-by: Kai Wasserbäch <kai at dev.carbon-project.org>
Acked-by: Lionel Landwerlin <lionel.g.landwerlin at intel.com>

---

 src/intel/common/gen_decoder.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/intel/common/gen_decoder.c b/src/intel/common/gen_decoder.c
index 69d7de0824e..97e8f168228 100644
--- a/src/intel/common/gen_decoder.c
+++ b/src/intel/common/gen_decoder.c
@@ -738,10 +738,10 @@ struct gen_spec *
 gen_spec_load_from_path(const struct gen_device_info *devinfo,
                         const char *path)
 {
-   size_t len, filename_len = strlen(path) + 20;
+   size_t filename_len = strlen(path) + 20;
    char *filename = malloc(filename_len);
 
-   len = snprintf(filename, filename_len, "%s/gen%i.xml",
+   ASSERTED size_t len = snprintf(filename, filename_len, "%s/gen%i.xml",
                   path, devinfo_to_gen(devinfo, false));
    assert(len < filename_len);
 




More information about the mesa-commit mailing list