File U_intel-decoder-mark-total_length-as-MAYBE_UNUS... of Package Mesa

Overview Repositories Revisions Requests Users Attributes Meta

File U_intel-decoder-mark-total_length-as-MAYBE_UNUSED-in-g.patch of Package Mesa (Revision 493c7a630f31e46b366087f15adb7258)

Currently displaying revision 493c7a630f31e46b366087f15adb7258 , Show latest

From 5fab32ddad8b32c4a05517c43623653f8634c4d9 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Kai=20Wasserb=C3=A4ch?= <kai@dev.carbon-project.org>
Date: Sat, 18 Aug 2018 13:16:15 +0200
Subject: [PATCH] intel/decoder: mark total_length as MAYBE_UNUSED in
 gen_spec_load
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Only used, when asserts are enabled.

Fixes an unused-variable warning with GCC 8:
 ../../../src/intel/common/gen_decoder.c: In function 'gen_spec_load':
 ../../../src/intel/common/gen_decoder.c:535:47: warning: variable 'total_length' set but not used [-Wunused-but-set-variable]
     uint32_t text_offset = 0, text_length = 0, total_length;
                                                ^~~~~~~~~~~~

Signed-off-by: Kai Wasserbäch <kai@dev.carbon-project.org>
Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
---
 src/intel/common/gen_decoder.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/intel/common/gen_decoder.c b/src/intel/common/gen_decoder.c
index ec0a486b10..c14c23aad1 100644
--- a/src/intel/common/gen_decoder.c
+++ b/src/intel/common/gen_decoder.c
@@ -532,7 +532,8 @@ gen_spec_load(const struct gen_device_info *devinfo)
    struct parser_context ctx;
    void *buf;
    uint8_t *text_data = NULL;
-   uint32_t text_offset = 0, text_length = 0, total_length;
+   uint32_t text_offset = 0, text_length = 0;
+   MAYBE_UNUSED uint32_t total_length;
    uint32_t gen_10 = devinfo_to_gen(devinfo);
 
    for (int i = 0; i < ARRAY_SIZE(genxml_files_table); i++) {
-- 
2.16.4

Places

File U_intel-decoder-mark-total_length-as-MAYBE_UNUSED-in-g.patch of Package Mesa (Revision 493c7a630f31e46b366087f15adb7258)

Places