babeld: address some compilation warnings

Including system headers is not necessary with zebra.h included and
sometimes results in "__ASSERT_FUNCTION redefined" compilation warning.

* babeld.c
  * babel_distribute_update_interface(): make static
* babel_interface.c
  * interface_config_write(): unused 'babel_ifp'
  * don't include system headers
* message.c
  * send_request(): unused 'babel_ifp'
  * send_multihop_request(): idem
  * don't include system headers
* route.c: don't include system headers
* xroute.c: idem
* source.h: newline at EOF
* message.h: idem

diff --git a/babeld/babel_interface.c b/babeld/babel_interface.c
index 0130f26b1a953392641971390de18ac81af26251..ff9c5eb473461e2257a2de130fe09a46fdffe37c 100644 (file)
--- a/babeld/babel_interface.c
+++ b/babeld/babel_interface.c
@@ -37,17 +37,6 @@ OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
 THE SOFTWARE.
 */
 
-#include <string.h>
-#include <stdio.h>
-#include <stdlib.h>
-#include <assert.h>
-#include <sys/time.h>
-#include <sys/ioctl.h>
-#include <sys/socket.h>
-#include <netinet/in.h>
-#include <net/if.h>
-#include <arpa/inet.h>
-
 #include <zebra.h>
 #include "memory.h"
 #include "log.h"
@@ -724,12 +713,9 @@ interface_config_write (struct vty *vty)
 {
     struct listnode *node;
     struct interface *ifp;
-    babel_interface_nfo *babel_ifp;
     int write = 0;
 
     for (ALL_LIST_ELEMENTS_RO (iflist, node, ifp)) {
-        babel_ifp = babel_get_if_nfo(ifp);
-
         /* Do not display the interface if there is no configuration about it */
         if (ifp->desc == NULL)
             continue;

diff --git a/babeld/babeld.c b/babeld/babeld.c
index 18b4e34f0354ad67497cdc14cfdc30c8eb857510..3dc35b5d55440b23fa3a0bea06d51412f44f76f2 100644 (file)
--- a/babeld/babeld.c
+++ b/babeld/babeld.c
@@ -560,7 +560,7 @@ babel_distribute_update (struct distribute *dist)
     }
 }
 
-void
+static void
 babel_distribute_update_interface (struct interface *ifp)
 {
     struct distribute *dist;

diff --git a/babeld/message.c b/babeld/message.c
index 57d875fbeb2d13abf777e87c9a05501ae044b18d..7930a1b13929b0349dba1b5069e5af9a9d53cd37 100644 (file)
--- a/babeld/message.c
+++ b/babeld/message.c
@@ -37,14 +37,6 @@ OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
 THE SOFTWARE.
 */
 
-#include <stdlib.h>
-#include <string.h>
-#include <stdio.h>
-#include <assert.h>
-#include <sys/time.h>
-#include <netinet/in.h>
-#include <arpa/inet.h>
-
 #include <zebra.h>
 #include "if.h"
 
@@ -1216,7 +1208,6 @@ void
 send_request(struct interface *ifp,
              const unsigned char *prefix, unsigned char plen)
 {
-    babel_interface_nfo *babel_ifp = NULL;
     int v4, len;
 
     if(ifp == NULL) {
@@ -1236,7 +1227,6 @@ send_request(struct interface *ifp,
     if(!if_up(ifp))
         return;
 
-    babel_ifp = babel_get_if_nfo(ifp);
     debugf(BABEL_DEBUG_COMMON,"sending request to %s for %s.",
            ifp->name, prefix ? format_prefix(prefix, plen) : "any");
     v4 = plen >= 96 && v4mapped(prefix);
@@ -1288,7 +1278,6 @@ send_multihop_request(struct interface *ifp,
                       unsigned short seqno, const unsigned char *id,
                       unsigned short hop_count)
 {
-    babel_interface_nfo *babel_ifp = NULL;
     int v4, pb, len;
 
     /* Make sure any buffered updates go out before this request. */
@@ -1308,7 +1297,6 @@ send_multihop_request(struct interface *ifp,
     if(!if_up(ifp))
         return;
 
-    babel_ifp = babel_get_if_nfo(ifp);
     debugf(BABEL_DEBUG_COMMON,"Sending request (%d) on %s for %s.",
            hop_count, ifp->name, format_prefix(prefix, plen));
     v4 = plen >= 96 && v4mapped(prefix);

diff --git a/babeld/route.c b/babeld/route.c
index a92018fd179b80d51275da66e20c950090902890..aa181be70bab3d8292abaa301b49cc79d51db365 100644 (file)
--- a/babeld/route.c
+++ b/babeld/route.c
@@ -37,13 +37,6 @@ OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
 THE SOFTWARE.
 */
 
-#include <stdlib.h>
-#include <string.h>
-#include <stdio.h>
-#include <errno.h>
-#include <assert.h>
-#include <sys/time.h>
-
 #include <zebra.h>
 #include "if.h"
 

diff --git a/babeld/xroute.c b/babeld/xroute.c
index 7b09cb8716d6c7aa5fb09f83c7a41085bc48b4ab..eb4aaeb570f59d2fa77b7a1e4cab4bcbe75bf172 100644 (file)
--- a/babeld/xroute.c
+++ b/babeld/xroute.c
@@ -37,14 +37,6 @@ OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
 THE SOFTWARE.
 */
 
-#include <stdlib.h>
-#include <stdio.h>
-#include <string.h>
-#include <errno.h>
-#include <assert.h>
-#include <sys/time.h>
-#include <netinet/in.h>
-
 #include <zebra.h>
 #include "if.h"
 #include "log.h"