You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@nuttx.apache.org by GitBox <gi...@apache.org> on 2022/01/21 11:46:31 UTC

[GitHub] [incubator-nuttx] a-lunev commented on a change in pull request #5296: tcp: fixed ISO C89/C90 related warnings

a-lunev commented on a change in pull request #5296:
URL: https://github.com/apache/incubator-nuttx/pull/5296#discussion_r789592118



##########
File path: net/tcp/tcp_accept.c
##########
@@ -232,7 +232,7 @@ int psock_tcp_accept(FAR struct socket *psock, FAR struct sockaddr *addr,
     {
       /* Yes... get the address of the connected client */
 
-      ninfo("Pending conn=%p\n", state.acpt_newconn);
+      ninfo("Pending conn=%p\n", (void *)state.acpt_newconn);

Review comment:
       If I compile the file by passing "-std=c89 -Wpedantic", the gcc produces "warning: format ‘%p’ expects argument of type ‘void *’, but argument 3 has type ‘struct tcp_conn_s *’ [-Wformat=]".




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org