[lwip-devel] [bug #34057] socklen_t should be a typedef.

[Brian McFarland]

URL:
  <http://savannah.nongnu.org/bugs/?34057>

                 Summary: socklen_t should be a typedef.
                 Project: lwIP - A Lightweight TCP/IP stack
            Submitted by: bmcfarland
            Submitted on: Thu 18 Aug 2011 02:06:35 PM GMT
                Category: sockets
                Severity: 3 - Normal
              Item Group: Change Request
                  Status: None
                 Privacy: Public
             Assigned to: None
             Open/Closed: Open
         Discussion Lock: Any
         Planned Release: 
            lwIP version: Other

    _______________________________________________________

Details:

It seems like "socklen_t", which is #defined in sockets.h should be a typedef,
not a #define.  I suggest this because a true typedef causes compilers to
produce better error messages.  

For example, I'm working on a port of thttpd and it was producing an error
"conflicting types for 'u32_t'".  The cause was that thttpd was trying to
redefine socklen_t, and so the real fix was in thttpd, not lwip.  However,
using a typedef produces the error: "conflicting types for 'socklen_t'", which
makes the problem more obvious.  

I'm using v1.3.2 out of the FreeRTOS source tree, but v1.4.0 still contains
the same #define.  Admittedly, I have not checked in CVS.




    _______________________________________________________

Reply to this item at:

  <http://savannah.nongnu.org/bugs/?34057>

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