[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5154: net/usrsock: replace xid type to uint64_t to avoid limitation of connections

[GitBox <gi...@apache.org>]

xiaoxiang781216 commented on a change in pull request #5154:
URL: https://github.com/apache/incubator-nuttx/pull/5154#discussion_r777940865



##########
File path: include/nuttx/net/usrsock.h
##########
@@ -227,9 +227,8 @@ begin_packed_struct struct usrsock_message_req_ack_s
 {
   struct usrsock_message_common_s head;
 
-  uint8_t xid;
-  uint8_t reserved;
-  int32_t result;
+  uint64_t xid;
+  int32_t  result;

Review comment:
       Change to:
   ```
     uint16_t reserved;
     int32_t result;
     uint64_t xid;
   ```
   to keep the alignment




-- 
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