Re: [PATCH] kvm: Avoid shadowing a local in search_memslots()

From: Qian Cai
Date: Tue Oct 26 2021 - 12:14:53 EST




On 10/26/21 11:44 AM, Sean Christopherson wrote:
> Maybe "pivot"? Or just "tmp"? I also vote to hoist the declaration out of the
> loop precisely to avoid potential shadows, and to also associate the variable
> with the "start" and "end" variables, e.g.

Actually, I am a bit more prefer to keep the declaration inside the loop
as it makes the declaration and assignment closer to make it easier to
understand the code. It should be relatively trivial to avoid potential
shadows in the future. It would be interesting to see what Paolo would say.