Change external functions to static functions, where possible.

Found by -Wmissing-prototypes.

diff --git a/lib/fault.c b/lib/fault.c
index 762357097d7dc6d5edd042ac02fbda2189d4aa27..5a53f15d898ecbde56161149feabbba99fe91ce0 100644 (file)
--- a/lib/fault.c
+++ b/lib/fault.c
@@ -1,4 +1,4 @@
-/* Copyright (c) 2008 The Board of Trustees of The Leland Stanford
+/* Copyright (c) 2008, 2009 The Board of Trustees of The Leland Stanford
  * Junior University
  * 
  * We are making the OpenFlow specification and associated documentation
@@ -44,7 +44,7 @@
 #include "vlog.h"
 #define THIS_MODULE VLM_fault
 
-void
+static void
 fault_handler(int sig_nr)
 {
     VLOG_EMER("Caught signal %d.", sig_nr);

diff --git a/lib/vconn-ssl.c b/lib/vconn-ssl.c
index a00906c963b73c0dd46240ef06d6327e784b5a8a..5dadc36365b59116b1e59c4e0401d21e44451eac 100644 (file)
--- a/lib/vconn-ssl.c
+++ b/lib/vconn-ssl.c
@@ -1,4 +1,4 @@
-/* Copyright (c) 2008 The Board of Trustees of The Leland Stanford
+/* Copyright (c) 2008, 2009 The Board of Trustees of The Leland Stanford
  * Junior University
  * 
  * We are making the OpenFlow specification and associated documentation
@@ -177,7 +177,7 @@ static void ssl_tx_poll_callback(int fd, short int revents, void *vconn_);
 static DH *tmp_dh_callback(SSL *ssl, int is_export UNUSED, int keylength);
 static void log_ca_cert(const char *file_name, X509 *cert);
 
-short int
+static short int
 want_to_poll_events(int want)
 {
     switch (want) {
@@ -1049,7 +1049,7 @@ vconn_ssl_set_certificate_file(const char *file_name)
  * in '*n_certs', and returns a positive errno value.
  *
  * The caller is responsible for freeing '*certs'. */
-int
+static int
 read_cert_file(const char *file_name, X509 ***certs, size_t *n_certs)
 {
     FILE *file;

diff --git a/tests/test-hmap.c b/tests/test-hmap.c
index 1ef6fea0d24ba095e1cc0d16a078186358c2b562..f4871e79a0569f387f1bd6187d0e503e66915abc 100644 (file)
--- a/tests/test-hmap.c
+++ b/tests/test-hmap.c
@@ -89,7 +89,7 @@ make_hmap(struct hmap *hmap, struct element elements[],
     }
 }
 
-void
+static void
 shuffle(int *p, size_t n)
 {
     for (; n > 1; n--, p++) {

diff --git a/udatapath/datapath.c b/udatapath/datapath.c
index e840045917f37eca595c26ec5395bf6061d75eaf..f5de7c5e429b7443d93131330c259655ba49847d 100644 (file)
--- a/udatapath/datapath.c
+++ b/udatapath/datapath.c
@@ -544,7 +544,7 @@ output_all(struct datapath *dp, struct ofpbuf *buffer, int in_port, int flood)
     return 0;
 }
 
-void
+static void
 output_packet(struct datapath *dp, struct ofpbuf *buffer, uint16_t out_port) 
 {
     struct sw_port *p = lookup_port(dp, out_port);

diff --git a/utilities/vlogconf.c b/utilities/vlogconf.c
index 6da0c1718169164a3abd9a7dbb4e0acbcae39422..c6a891a4f914f6e036f78dc6e31e6b3358c66aeb 100644 (file)
--- a/utilities/vlogconf.c
+++ b/utilities/vlogconf.c
@@ -1,4 +1,4 @@
-/* Copyright (c) 2008 The Board of Trustees of The Leland Stanford
+/* Copyright (c) 2008, 2009 The Board of Trustees of The Leland Stanford
  * Junior University
  * 
  * We are making the OpenFlow specification and associated documentation
@@ -47,7 +47,7 @@
 #include "util.h"
 #include "vlog-socket.h"
 
-void
+static void
 usage(char *prog_name, int exit_code)
 {
     printf("Usage: %s [TARGET] [ACTION...]\n"

diff --git a/vswitchd/brcompat.c b/vswitchd/brcompat.c
index 0151d1ff2fa361b47c119687197fba5e0591ead3..4a877ffa06b40429940940b469863204727fac63 100644 (file)
--- a/vswitchd/brcompat.c
+++ b/vswitchd/brcompat.c
@@ -1,4 +1,4 @@
-/* Copyright (c) 2008 The Board of Trustees of The Leland Stanford
+/* Copyright (c) 2008, 2009 The Board of Trustees of The Leland Stanford
  * Junior University
  *
  * We are making the OpenFlow specification and associated documentation
@@ -309,7 +309,7 @@ brc_del_dp(const char *dp_name)
     return 0;
 }
 
-int 
+static int 
 brc_handle_dp_cmd(struct ofpbuf *buffer, bool add)
 {
     int dp_act_err;
@@ -350,7 +350,7 @@ static const struct nl_policy brc_port_policy[] = {
     [BRC_GENL_A_PORT_NAME] = { .type = NL_A_STRING },
 };
 
-int 
+static int 
 brc_handle_port_cmd(struct ofpbuf *buffer, bool add)
 {
     struct nlattr *attrs[ARRAY_SIZE(brc_port_policy)];
@@ -381,7 +381,7 @@ brc_handle_port_cmd(struct ofpbuf *buffer, bool add)
     return 0;
 }
 
-int 
+static int 
 brc_recv_update(void)
 {
     int retval;

diff --git a/vswitchd/bridge.c b/vswitchd/bridge.c
index 8f6cc6e377e541b40cd0e5d15502610f25c8776e..73dd3d2125b1afc1929c87bfb4157fb3d3613d2a 100644 (file)
--- a/vswitchd/bridge.c
+++ b/vswitchd/bridge.c
@@ -220,6 +220,7 @@ static void bridge_reconfigure_one(struct bridge *);
 static void bridge_get_all_ifaces(const struct bridge *, struct svec *ifaces);
 static bool bridge_is_backlogged(const struct bridge *);
 static int bridge_fetch_dp_ifaces(struct bridge *, struct svec *iface_names);
+static void bridge_flush(struct bridge *);
 
 static void bridge_process_msg(struct bridge *, struct ofpbuf *);
 static void revalidate_flow(struct bridge *, struct ft_flow *);
@@ -438,7 +439,7 @@ bridge_wait(void)
 
 /* Forces 'br' to revalidate all of its flows.  This is appropriate when 'br''s
  * configuration changes.  */
-void
+static void
 bridge_flush(struct bridge *br)
 {
     br->flush = true;