Bruno Jesus 00cpxxx@gmail.com wrote:
else if(a->ai_canonname || b->ai_canonname)
ok(0, "Expected both canonical names present (%p != %p)\n", a->ai_canonname, b->ai_canonname);
It would look more natural IMHO to use ok(!a->ai_canonname && !b->ai_canonname, "Expected both canonical names present (%p != %p)\n", a->ai_canonname, b->ai_canonname); instead of explicit ok(0) (here and in the not unicode variant).
Indeed, much better.
Please take into account my typos above (remove '!'s).