[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[lwip-devel] [patch #9447] ping: guard ping_time with LWIP_DEBUG

From: Joel Cunningham
Subject: [lwip-devel] [patch #9447] ping: guard ping_time with LWIP_DEBUG
Date: Mon, 28 Aug 2017 13:32:27 -0400 (EDT)
User-agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:55.0) Gecko/20100101 Firefox/55.0


                 Summary: ping: guard ping_time with LWIP_DEBUG
                 Project: lwIP - A Lightweight TCP/IP stack
            Submitted by: jcunningham
            Submitted on: Mon 28 Aug 2017 05:32:26 PM UTC
                Category: apps
                Priority: 5 - Normal
                  Status: None
                 Privacy: Public
             Assigned to: jcunningham
        Originator Email: 
             Open/Closed: Open
         Discussion Lock: Any
         Planned Release: None



Attached patch, commit message:

ping_time is only used in LWIP_DEBUGF statements.  When LWIP_DEBUG is
not defined (and by extension LWIP_DEBUGF is a no-op), this causes
compiler warning 550 with the ARMCC compiler:

 'ping_time was set but never used'

Just wanted to double check #ifdef LWIP_DEBUG is the correct guard for
ping_time before submitting this change.  Reviews appreciated :)


File Attachments:

Date: Mon 28 Aug 2017 05:32:26 PM UTC  Name:
0001-ping-guard-ping_time-with-LWIP_DEBUG.patch  Size: 1KiB   By: jcunningham



Reply to this item at:


  Message sent via/by Savannah

reply via email to

[Prev in Thread] Current Thread [Next in Thread]