This is an automated email from the ASF dual-hosted git repository.

xiaoxiang781216 pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/nuttx-apps.git

commit da92a492bda0d58534de97a66cfde0db06705244
Author: raiden00pl <[email protected]>
AuthorDate: Thu Oct 8 10:40:14 2026 +0200

    netutils/dhcpd: Replace the FIXME #warning with a comment.
    
    arm64 builds C code with -Werror, so the #warning in
    dhcpd_allocipaddr() made netutils/dhcpd fail to build on every arm64
    target. Keep the note as a comment.
    
    Assisted-by: Claude:claude-opus-5-5
    Signed-off-by: raiden00pl <[email protected]>
---
 netutils/dhcpd/dhcpd.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/netutils/dhcpd/dhcpd.c b/netutils/dhcpd/dhcpd.c
index d00d597f3..61842ce26 100644
--- a/netutils/dhcpd/dhcpd.c
+++ b/netutils/dhcpd/dhcpd.c
@@ -517,10 +517,10 @@ static in_addr_t dhcpd_allocipaddr(void)
       lease = dhcpd_findbyipaddr(ipaddr);
       if ((!lease || dhcpd_leaseexpired(lease)))
         {
-#ifdef CONFIG_CPP_HAVE_WARNING
-#  warning "FIXME: Should check if anything responds to an ARP request or ping"
-#  warning "       to verify that there is no other user of this IP address"
-#endif
+          /* FIXME: Should check if anything responds to an ARP request or
+           * ping to verify that there is no other user of this IP address.
+           */
+
           memset(g_state.ds_leases[ipaddr - startaddr].mac,
           0, DHCP_HLEN_ETHERNET);
           g_state.ds_leases[ipaddr - startaddr].allocated = true;

Reply via email to