vlog: Stop using explicit references to external log modules.
authorBen Pfaff <blp@ovn.org>
Wed, 3 Feb 2016 21:15:48 +0000 (13:15 -0800)
committerBen Pfaff <blp@ovn.org>
Wed, 3 Feb 2016 23:20:11 +0000 (15:20 -0800)
commit45863ce567c8edf53b5e41e359295f694bb9d3b4
treecf4d1bca777ae80bcaa0c5f78a32febcc992f4c0
parenteba7755248f3c042238cf319676dead0dcc785c6
vlog: Stop using explicit references to external log modules.

It's always risky to write "extern" declarations outside a header file,
since there's no way to ensure the type of what's being referenced is
correct.  In these cases, we can easily avoid the extern reference, so do
so.

There is a little tradeoff here, in that referring to the log modules
through strings means that we catch an incorrect module name at runtime
instead of at link time, but I think that the risk here is minimal because
the mistake will be found by every test in "make check" that runs any of
the utilities, since they make these calls as one of their first tasks
during initialization.

Signed-off-by: Ben Pfaff <blp@ovn.org>
Acked-by: Russell Bryant <russell@ovn.org>
ovn/utilities/ovn-nbctl.c
ovn/utilities/ovn-sbctl.c
tests/test-reconnect.c
utilities/ovs-vsctl.c
vtep/vtep-ctl.c