Re: [PATCH] test-throttle: don't shadow 'index' variable in do_test_accounting()

[Markus Armbruster]

Alberto Garcia  writes:

> Fixes build with -Wshadow=local
>
> Signed-off-by: Alberto Garcia 
> ---
>  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 
and queued, thanks!

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

[Alberto Garcia]

Fixes build with -Wshadow=local

Signed-off-by: Alberto Garcia 
---
 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;
 }
 
-- 
2.39.2