[lwip-devel] [bug #28659] Missing casts: compile error when using C++ OS headers

[Simon Goldschmidt]

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

                 Summary: Missing casts: compile error when using C++ OS
headers
                 Project: lwIP - A Lightweight TCP/IP stack
            Submitted by: goldsimon
            Submitted on: Mo 18 Jan 2010 07:40:26 GMT
                Category: None
                Severity: 3 - Normal
              Item Group: Compiler Warning
                  Status: Need Info
                 Privacy: Public
             Assigned to: None
             Open/Closed: Open
         Discussion Lock: Any
         Planned Release: 
            lwIP version: CVS Head

    _______________________________________________________

Details:

I'm currently at a project where we use C++ OS headers (for the sys_arch.h
defines). Since these defines are replaced by actual C++ code (e.g.
SYS_ARCH_PROTECT is replaced by OS::Protect()), I need to compile the C files
as C++-code.

There are many warnings I can turn off, but at least under MS-VS, missing
casts (e.g. from void* to struct eth_hdr*) are an error (not a warning), which
can't be turned off.

I know that the casts are not required in C (but still allowed), but there's
no way to compile the code using my OS'es defines like it is now: I'd have to
switch to using (slow) functions instead of defines.

So the question is: Does anyone have anything against me adding these casts?




    _______________________________________________________

Reply to this item at:

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

_______________________________________________
  Nachricht geschickt von/durch Savannah
  http://savannah.nongnu.org/