diff options
| author | Renato Westphal <renato@cvs.openbsd.org> | 2016-09-02 16:44:34 +0000 |
|---|---|---|
| committer | Renato Westphal <renato@cvs.openbsd.org> | 2016-09-02 16:44:34 +0000 |
| commit | fcf9752bb7cfa0955234368e43600610c4308242 (patch) | |
| tree | f12e2cdd668aab5d8339150fa1af1dfada1dda77 /usr.sbin/eigrpd/reply.c | |
| parent | dcaa3316f517f3e75debd2dc1ae2838de28a7b02 (diff) | |
Make functions and variables static whenever possible.
style(9) says:
"Function prototypes for private functions (i.e., functions not used
elsewhere) go at the top of the first source module. In userland,
functions local to one source module should be declared 'static'".
The benefits of doing so include:
* clean up of the eigrpd global namespace;
* improved readability;
* more hints to the compiler/linker to generate more efficient code.
Additional changes:
* Declare all extern variables in header files;
* Clean up the indentation of all function prototypes and global
variables.
ok claudio@ benno@
Diffstat (limited to 'usr.sbin/eigrpd/reply.c')
| -rw-r--r-- | usr.sbin/eigrpd/reply.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/usr.sbin/eigrpd/reply.c b/usr.sbin/eigrpd/reply.c index 9e4ee903480..f12f6dae10a 100644 --- a/usr.sbin/eigrpd/reply.c +++ b/usr.sbin/eigrpd/reply.c @@ -1,4 +1,4 @@ -/* $OpenBSD: reply.c,v 1.3 2016/09/02 16:29:55 renato Exp $ */ +/* $OpenBSD: reply.c,v 1.4 2016/09/02 16:44:33 renato Exp $ */ /* * Copyright (c) 2015 Renato Westphal <renato@openbsd.org> @@ -27,8 +27,6 @@ #include "eigrpe.h" #include "log.h" -extern struct eigrpd_conf *econf; - /* reply packet handling */ void |