18 Feb
2014
18 Feb
'14
4:06 a.m.
Bruno Jesus <00cpxxx(a)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). -- Dmitry.