Re: [PATCH 0/8] MR506: configure: Enable -Wshadow.

[Giovanni Mascellani (＠giomasce)]

Giovanni Mascellani (@giomasce) commented about tests/d3d12.c:
>              check_copyable_footprints(&resource_desc, 0, sub_resource_count, base_offsets[k],
>                      NULL, NULL, NULL, &total_size);
>
> -
>            for (l = 0; l < sub_resource_count; ++l)
> -
>            for (unsigned int l = 0; l < sub_resource_count; ++l)

I always thought we don't like to declare variables in the `for` header, and while I don't necessarily agree with that rule I think that there is some rule in keeping things consistent.

I like the MR overall.