diff options
| author | Ben Pfaff <blp@ovn.org> | 2016-02-03 13:21:10 -0800 |
|---|---|---|
| committer | Ben Pfaff <blp@ovn.org> | 2016-02-10 13:35:56 -0800 |
| commit | 922fed065e65ad8388528976d631445f9e768f55 (patch) | |
| tree | 4ce22727b222aed78846fae48a2ae046a7b8d2fb /lib/vconn-stream.c | |
| parent | 3ca6cc76c3f3d171fcc7361a7ad4edd452834b7c (diff) | |
| download | openvswitch-922fed065e65ad8388528976d631445f9e768f55.tar.gz | |
vlog: Make the most common module reference more direct.
Most vlog calls are for the log module owned by the translation unit being
compiled, but this module was referenced indirectly through a pointer
variable. That seems silly, so this commit changes the code so that the
local vlog module is referred to directly, as &this_module.
We could get rid of the global variables for vlog modules entirely, but
I like getting linker errors when there's a duplicate module name.
Signed-off-by: Ben Pfaff <blp@ovn.org>
Acked-by: Russell Bryant <russell@ovn.org>
Diffstat (limited to 'lib/vconn-stream.c')
| -rw-r--r-- | lib/vconn-stream.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/lib/vconn-stream.c b/lib/vconn-stream.c index 23c0aaecd..2858462be 100644 --- a/lib/vconn-stream.c +++ b/lib/vconn-stream.c @@ -1,5 +1,5 @@ /* - * Copyright (c) 2008, 2009, 2010, 2011, 2012, 2013 Nicira, Inc. + * Copyright (c) 2008, 2009, 2010, 2011, 2012, 2013, 2016 Nicira, Inc. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -104,7 +104,7 @@ vconn_stream_close(struct vconn *vconn) if ((vconn->error == EPROTO || s->n_packets < 1) && s->rxbuf) { stream_report_content(s->rxbuf->data, s->rxbuf->size, STREAM_OPENFLOW, - THIS_MODULE, vconn_get_name(vconn)); + &this_module, vconn_get_name(vconn)); } stream_close(s->stream); |