diff mbox series

test-throttle: don't shadow 'index' variable in do_test_accounting()

Message ID 20230922105742.81317-1-berto@igalia.com
State New
Headers show
Series test-throttle: don't shadow 'index' variable in do_test_accounting() | expand

Commit Message

Alberto Garcia Sept. 22, 2023, 10:57 a.m. UTC 
Fixes build with -Wshadow=local

Signed-off-by: Alberto Garcia <berto@igalia.com>
---
 tests/unit/test-throttle.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Markus Armbruster Sept. 29, 2023, 5:40 a.m. UTC | #1 
Alberto Garcia <berto@igalia.com> writes:

> Fixes build with -Wshadow=local
>
> Signed-off-by: Alberto Garcia <berto@igalia.com>
> ---
>  tests/unit/test-throttle.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tests/unit/test-throttle.c b/tests/unit/test-throttle.c
> index cb587e33e7..ac35d65d19 100644
> --- a/tests/unit/test-throttle.c
> +++ b/tests/unit/test-throttle.c
> @@ -625,7 +625,7 @@ static bool do_test_accounting(bool is_ops, /* are we testing bps or ops */
>      throttle_config_init(&cfg);
>  
>      for (i = 0; i < 3; i++) {
> -        BucketType index = to_test[is_ops][i];
> +        index = to_test[is_ops][i];
>          cfg.buckets[index].avg = avg;
>      }

Reviewed-by: Markus Armbruster <armbru@redhat.com>
and queued, thanks!
diff mbox series

Patch

diff --git a/tests/unit/test-throttle.c b/tests/unit/test-throttle.c
index cb587e33e7..ac35d65d19 100644
--- a/tests/unit/test-throttle.c
+++ b/tests/unit/test-throttle.c
@@ -625,7 +625,7 @@  static bool do_test_accounting(bool is_ops, /* are we testing bps or ops */
     throttle_config_init(&cfg);
 
     for (i = 0; i < 3; i++) {
-        BucketType index = to_test[is_ops][i];
+        index = to_test[is_ops][i];
         cfg.buckets[index].avg = avg;
     }