Clang 3.9 report

Julien ÉLIE julien at trigofacile.com
Fri Aug 26 19:18:30 UTC 2016


Hi The Doctor,

> lib/getaddrinfo-t.c:265:13: warning: logical not is only applied to the left
>       hand side of this comparison [-Wlogical-not-parentheses]
> 	      if (!strcmp(ai->ai_canonname, first->ai_canonname) == 0) {

Thanks for the report.
I would suggest the following change:

--- getaddrinfo-t.c	(révision 10040)
+++ getaddrinfo-t.c	(copie de travail)
@@ -262,7 +262,7 @@
      is_int(0, saddr->sin_port, "...port is 0");
      first = ai;
      for (found = 0; ai != NULL; ai = ai->ai_next) {
-        if (!strcmp(ai->ai_canonname, first->ai_canonname) == 0) {
+        if (strcmp(ai->ai_canonname, first->ai_canonname) != 0) {
              ok(0, "...canonname matches");
              break;
          }


Russ, could you take it into account in rra-c-util?
Thanks,

-- 
Julien ÉLIE

« Tu dis que tu aimes les fleurs et tu leur coupes la queue, tu dis
   que tu aimes les chiens et tu leur mets une laisse, tu dis que tu
   aimes les oiseaux et tu les mets en cage, tu dis que tu m'aimes alors
   moi j'ai peur. » (Jean Cocteau)


More information about the inn-workers mailing list