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

[Joel Cunningham]

URL:
  <http://savannah.nongnu.org/patch/?9447>

                 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

    _______________________________________________________

Details:

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

<http://savannah.nongnu.org/patch/download.php?file_id=41689>

    _______________________________________________________

Reply to this item at:

  <http://savannah.nongnu.org/patch/?9447>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.nongnu.org/