iphlpapi: Avoid sockaddr scope confusion

Bruno Jesus 00cpxxx at gmail.com
Wed Jan 11 14:56:34 CST 2017

On Wed, Jan 11, 2017 at 6:51 PM, Alexandre Julliard <julliard at winehq.org> wrote:
> Bruno Jesus <00cpxxx at gmail.com> writes:
>> On Wed, Jan 11, 2017 at 6:14 PM, Alexandre Julliard <julliard at winehq.org> wrote:
>>> André Hentschel <nerv at dawncrow.de> writes:
>>>> Signed-off-by: André Hentschel <nerv at dawncrow.de>
>>>> ---
>>>>  dlls/iphlpapi/iphlpapi_main.c | 2 +-
>>>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>> This doesn't look to me like an improvement.
>> It is an improvement because a same name variable is declared inside a
>> for [2] lines below this one. So reducing the scope will make the
>> variables no longer overlap.
> It's a different scope, they already don't overlap. It's done the same
> way as the v4 case above it.

You are right, they are not in the same scope. Yet it is not exactly
like ipv4 case. Line 1116 declares the prefix and line 1122 the sa,
while in the ipv6 case both variables are declared in 1078. The change
makes the code look more similar by declaring the ipv6 sa inside the
for just like the ipv4.

More information about the wine-devel mailing list