| Message ID | 20230922105742.81317-1-berto@igalia.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | test-throttle: don't shadow 'index' variable in do_test_accounting() | expand |
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 --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; }
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(-)