compare-framebuffer-parameter-with-get.c:143:14: warning: variable 'found' is 
used uninitialized whenever 'for' loop exits because its condition is false 
[-Wsometimes-uninitialized]
        for (i = 0; i < ARRAY_SIZE(table_23_73_allowed); i++) {
                    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
compare-framebuffer-parameter-with-get.c:150:7: note: uninitialized use occurs 
here
        if (!found) {
             ^~~~~
compare-framebuffer-parameter-with-get.c:143:14: note: remove the condition if 
it is always true
        for (i = 0; i < ARRAY_SIZE(table_23_73_allowed); i++) {
                    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
compare-framebuffer-parameter-with-get.c:130:12: note: initialize the variable 
'found' to silence this warning
        bool found;
                  ^
                   = false

Fixes: d60f1fb839bc ("gl-4.5/compare-framebuffer-parameter-with-get: add test")
Signed-off-by: Vinson Lee <[email protected]>
---
 tests/spec/gl-4.5/compare-framebuffer-parameter-with-get.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/spec/gl-4.5/compare-framebuffer-parameter-with-get.c 
b/tests/spec/gl-4.5/compare-framebuffer-parameter-with-get.c
index b10a780b70ac..a5985bfcd1e7 100644
--- a/tests/spec/gl-4.5/compare-framebuffer-parameter-with-get.c
+++ b/tests/spec/gl-4.5/compare-framebuffer-parameter-with-get.c
@@ -127,7 +127,7 @@ static void
 parse_args(int argc, char **argv)
 {
        int i;
-       bool found;
+       bool found = false;
 
        if (argc > 3) {
                printf("Only two possible params supported\n");
-- 
2.9.3

_______________________________________________
Piglit mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/piglit

Reply via email to