[zebra] fix inconsistencies in ifstat_update_* declarations and definitions
2006-08-06 Paul Jakma <paul.jakma@sun.com>
* interface.h: (ifstat_update_proc) declaration should match
ifstat_update_sysctl really, which is to not return status, as
such status is not used anywhere.
* if_{proc,sysctl}.c: Make ifstat_update_* definitions and return values
consistent with each other and their declarations, ie:
(void) (*) (void).
diff --git a/zebra/if_proc.c b/zebra/if_proc.c
index 6002595..367b0d6 100644
--- a/zebra/if_proc.c
+++ b/zebra/if_proc.c
@@ -123,8 +123,8 @@
}
/* Update interface's statistics. */
-int
-ifstat_update_proc ()
+void
+ifstat_update_proc (void)
{
FILE *fp;
char buf[PROCBUFSIZ];
@@ -139,7 +139,7 @@
{
zlog_warn ("Can't open proc file %s: %s",
_PATH_PROC_NET_DEV, safe_strerror (errno));
- return -1;
+ return;
}
/* Drop header lines. */
@@ -162,7 +162,7 @@
ifstat_dev_fields (version, stat, ifp);
}
fclose(fp);
- return 0;
+ return;
}
/* Interface structure allocation by proc filesystem. */