Re: [PATCH v3] ws2_32:Succeed IPV6_V6ONLY setsockopt for AF_INET sockets ignoring it

[Dmitry Timoshkov]

Anton Romanov <theli.ua(a)gmail.com> wrote:
> +todo_wine {
> +    enabled = 2;
> +    ret = getsockopt(v4, IPPROTO_IPV6, IPV6_V6ONLY, (char*)&enabled, &len);
> +    ok(!ret, "getsockopt(IPV6_ONLY) failed (LastError: %d)\n", WSAGetLastError());
> +    ok(enabled == 1, "expected 1, got %d\n", enabled);
> +}

The 'todo_wine' statements should be added only around ok() call that will fail
otherwise. This applies for other places in the patch as well.

-- 
Dmitry.

[PATCH v3] ws2_32:Succeed IPV6_V6ONLY setsockopt for AF_INET sockets ignoring it

[Anton Romanov]

Those OK fail in wine.

On Jan 12, 2017 18:44, "Dmitry Timoshkov" <dmitry(a)baikal.ru> wrote:
> Anton Romanov <theli.ua(a)gmail.com> wrote:
> > +todo_wine {
> > +    enabled = 2;
> > +    ret = getsockopt(v4, IPPROTO_IPV6, IPV6_V6ONLY, (char*)&enabled,
> > &len);
> > +    ok(!ret, "getsockopt(IPV6_ONLY) failed (LastError: %d)\n",
> > WSAGetLastError());
> > +    ok(enabled == 1, "expected 1, got %d\n", enabled);
> > +}
>
> The 'todo_wine' statements should be added only around ok() call that will
> fail
> otherwise. This applies for other places in the patch as well.
>
> --
> Dmitry.

[PATCH v3] ws2_32:Succeed IPV6_V6ONLY setsockopt for AF_INET sockets ignoring it

[Dmitry Timoshkov]

Anton Romanov <theli.ua(a)gmail.com> wrote:
> > > +todo_wine {
> > > +    enabled = 2;
> > > +    ret = getsockopt(v4, IPPROTO_IPV6, IPV6_V6ONLY, (char*)&enabled,
> > > &len);
> > > +    ok(!ret, "getsockopt(IPV6_ONLY) failed (LastError: %d)\n",
> > > WSAGetLastError());
> > > +    ok(enabled == 1, "expected 1, got %d\n", enabled);
> > > +}
> >
> > The 'todo_wine' statements should be added only around ok() call that will
> > fail
> > otherwise. This applies for other places in the patch as well.
> Those OK fail in wine.

Of course, but my comment was about adding todo_wine only around the ok()
calls and nothing else.

-- 
Dmitry.

[PATCH v3] ws2_32:Succeed IPV6_V6ONLY setsockopt for AF_INET sockets ignoring it

[Anton Romanov]

On Thu, Jan 12, 2017 at 6:52 PM, Dmitry Timoshkov <dmitry(a)baikal.ru> wrote:
> Anton Romanov <theli.ua(a)gmail.com> wrote:
> > > > +todo_wine {
> > > > +    enabled = 2;
> > > > +    ret = getsockopt(v4, IPPROTO_IPV6, IPV6_V6ONLY, (char*)&enabled,
> > > > &len);
> > > > +    ok(!ret, "getsockopt(IPV6_ONLY) failed (LastError: %d)\n",
> > > > WSAGetLastError());
> > > > +    ok(enabled == 1, "expected 1, got %d\n", enabled);
> > > > +}
> > >
> > > The 'todo_wine' statements should be added only around ok() call that will
> > > fail
> > > otherwise. This applies for other places in the patch as well.
> > Those OK fail in wine.
>
> Of course, but my comment was about adding todo_wine only around the ok()
> calls and nothing else.

Well, existing tests in the same file wrap whole sections of the code
with wine_todo and I followed that.

[PATCH v3] ws2_32:Succeed IPV6_V6ONLY setsockopt for AF_INET sockets ignoring it

[Dmitry Timoshkov]

Anton Romanov <theli.ua(a)gmail.com> wrote:
> > > > > +todo_wine {
> > > > > +    enabled = 2;
> > > > > +    ret = getsockopt(v4, IPPROTO_IPV6, IPV6_V6ONLY, (char*)&enabled,
> > > > > &len);
> > > > > +    ok(!ret, "getsockopt(IPV6_ONLY) failed (LastError: %d)\n",
> > > > > WSAGetLastError());
> > > > > +    ok(enabled == 1, "expected 1, got %d\n", enabled);
> > > > > +}
> > > >
> > > > The 'todo_wine' statements should be added only around ok() call that will
> > > > fail
> > > > otherwise. This applies for other places in the patch as well.
> > > Those OK fail in wine.
> >
> > Of course, but my comment was about adding todo_wine only around the ok()
> > calls and nothing else.
>
> Well, existing tests in the same file wrap whole sections of the code
> with wine_todo and I followed that.

Thanks for the explanation, not really your fault then :)

-- 
Dmitry.