nuttxs commented on code in PR #3131:
URL: https://github.com/apache/nuttx-apps/pull/3131#discussion_r2204495848


##########
netutils/dhcpc/dhcpc.c:
##########
@@ -950,3 +968,107 @@ int dhcpc_request_async(FAR void *handle, 
dhcpc_callback_t callback)
 
   return OK;
 }
+
+/****************************************************************************
+ * Name: dhcpc_release
+ ****************************************************************************/
+
+int dhcpc_release(FAR void *handle, FAR struct dhcpc_state *presult)
+{
+  FAR struct dhcpc_state_s *pdhcpc = (FAR struct dhcpc_state_s *)handle;
+  int ret;
+  int send_attempts = 0;
+  const int max_attempts = CONFIG_NETUTILS_DHCPC_RELEASE_RETRIES;

Review Comment:
   I had a misunderstanding earlier. Using the macro directly, without 
intermediate variables, is more concise, and this has been updated.



-- 
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-unsubscr...@nuttx.apache.org

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

Reply via email to