Skip to content

Commit

Permalink
lib: Clean up vlog use.
Browse files Browse the repository at this point in the history
Vlog functions assume a vlog module has been defined for the current
translation unit.  Including lib/vlog.h from a header file makes the
vlog API visible even when no vlog module may not have been defined.
This patch removes the two cases in the tree where vlog.h was included
from a header file.

Signed-off-by: Jarno Rajahalme <jrajahalme@nicira.com>
Acked-by: Ben Pfaff <blp@nicira.com>
  • Loading branch information
Jarno Rajahalme committed Oct 24, 2014
1 parent 3a17f00 commit e618b42
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
1 change: 0 additions & 1 deletion lib/coverage.h
Expand Up @@ -28,7 +28,6 @@
* a useful debugging tool. */

#include "ovs-thread.h"
#include "vlog.h"
#include "compiler.h"

/* Makes coverage_run run every 5000 ms (5 seconds).
Expand Down
3 changes: 2 additions & 1 deletion lib/stream.h
Expand Up @@ -22,11 +22,12 @@
#include <stdint.h>
#include <sys/types.h>
#include "openvswitch/types.h"
#include "vlog.h"
#include "socket-util.h"
#include "util.h"

struct pstream;
struct stream;
struct vlog_module;

void stream_usage(const char *name, bool active, bool passive, bool bootstrap);

Expand Down

0 comments on commit e618b42

Please sign in to comment.