Wireshark mailing list archives

Re: -Wmissing-prototypes ??


From: Evan Huus <eapache () gmail com>
Date: Tue, 12 Mar 2013 23:07:22 -0400

The C++ standard requires prototypes for all functions, so g++
replaces gcc's -Wmissing-prototypes with a mandatory compiler error. I
assume that this was part of the C++ compatibility work that Anders
has been doing recently.

Evan

On Tue, Mar 12, 2013 at 10:55 PM, Bill Meier <wmeier () newsguy com> wrote:
Anders:

Just out of curiosity, why are the prototypes in the following change needed
?


Or: maybe the real question is: why is -Wmissing_prototypes needed ?

We already catch any .h files which are missing prototypes of global
functions because we use -Wimplicit-function-declaration (part of -Wall).

So: what am i missing ?

Bill




User: etxrab
Date: 2013/03/12 04:09 PM

Log:
  - [-Wmissing-prototypes]

--- packet-aodv.c       (revision 48273)
+++ packet-aodv.c       (revision 48274)
@@ -50,6 +50,8 @@
  *     (both of the above two are draft-perkins-manet-aodv6-01.txt, which
  *     is from November 2000)
  */
+void proto_register_aodv(void);
+void proto_reg_handoff_aodv(void);

 #define INET6_ADDRLEN  16
 #define UDP_PORT_AODV  654

___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <wireshark-dev () wireshark org>
Archives:    http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
            mailto:wireshark-dev-request () wireshark org?subject=unsubscribe
___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <wireshark-dev () wireshark org>
Archives:    http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
             mailto:wireshark-dev-request () wireshark org?subject=unsubscribe


Current thread: