On Fri, Nov 19, 2021 at 12:59:40PM +0000, Tvrtko Ursulin wrote: > From: Tvrtko Ursulin <[email protected]> > > .../tests/i915/api_intel_allocator.c: In function ‘basic_alloc’: > .../tests/i915/api_intel_allocator.c:158:25: warning: this ‘if’ clause does > not guard... [-Wmisleading-indentation] > 158 | if (j == i) > | ^~ > > Signed-off-by: Tvrtko Ursulin <[email protected]> > Cc: Zbigniew Kempczyński <[email protected]>
I've already seen this on gcc 11.2.0 (Ubuntu 21.10) and got locally same fix, so: Reviewed-by: Zbigniew Kempczyński <[email protected]> -- Zbigniew > --- > tests/i915/api_intel_allocator.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/tests/i915/api_intel_allocator.c > b/tests/i915/api_intel_allocator.c > index 4b74317ed3d0..6d7764ca3f51 100644 > --- a/tests/i915/api_intel_allocator.c > +++ b/tests/i915/api_intel_allocator.c > @@ -157,7 +157,7 @@ static void basic_alloc(int fd, int cnt, uint8_t type) > for (j = 0; j < cnt; j++) { > if (j == i) > continue; > - igt_assert(!overlaps(&obj[i], &obj[j])); > + igt_assert(!overlaps(&obj[i], &obj[j])); > } > } > > -- > 2.32.0 >
