[ovs-dev] [PATCH] vconn-stream: Make definitions of tcp_vconn_class, etc., easier to find.
Ben Pfaff
blp at nicira.com
Tue Feb 15 23:05:48 UTC 2011
Thanks. I pushed this commit.
On Tue, Feb 15, 2011 at 02:50:21PM -0800, Ethan Jackson wrote:
> Looks Good.
>
> Ethan
>
> On Fri, Feb 4, 2011 at 3:10 PM, Ben Pfaff <blp at nicira.com> wrote:
> > When object definitions are entirely hidden inside macros with token
> > pasting tricks, tools like "tags" and even "grep" cannot find them. This
> > change makes the definitions of tcp_vconn_class and other stream-based
> > vconns easier to find, by exposing their definitions to these tools.
> >
> > Reported-by: Yongqiang Liu <liuyq7809 at gmail.com>
> > ---
> > AUTHORS | 1 +
> > lib/vconn-stream.c | 34 +++++++++++++++++-----------------
> > 2 files changed, 18 insertions(+), 17 deletions(-)
> >
> > diff --git a/AUTHORS b/AUTHORS
> > index c48479f..6687b41 100644
> > --- a/AUTHORS
> > +++ b/AUTHORS
> > @@ -75,6 +75,7 @@ Stephen Hemminger shemminger at vyatta.com
> > Takayuki HAMA t-hama at cb.jp.nec.com
> > Teemu Koponen koponen at nicira.com
> > Vishal Swarankar vishal.swarnkar at gmail.com
> > +Yongqiang Liu liuyq7809 at gmail.com
> > kk yap yapkke at stanford.edu
> >
> > Thanks to all Open vSwitch contributors. If you are not listed above
> > diff --git a/lib/vconn-stream.c b/lib/vconn-stream.c
> > index 82854e9..39f1c94 100644
> > --- a/lib/vconn-stream.c
> > +++ b/lib/vconn-stream.c
> > @@ -1,5 +1,5 @@
> > /*
> > - * Copyright (c) 2008, 2009, 2010 Nicira Networks.
> > + * Copyright (c) 2008, 2009, 2010, 2011 Nicira Networks.
> > *
> > * Licensed under the Apache License, Version 2.0 (the "License");
> > * you may not use this file except in compliance with the License.
> > @@ -365,9 +365,9 @@ pvconn_pstream_wait(struct pvconn *pvconn)
> >
> > /* Stream-based vconns and pvconns. */
> >
> > -#define DEFINE_VCONN_STREAM_CLASS(NAME) \
> > - struct vconn_class NAME##_vconn_class = { \
> > - #NAME, \
> > +#define STREAM_INIT(NAME) \
> > + { \
> > + NAME, \
> > vconn_stream_open, \
> > vconn_stream_close, \
> > vconn_stream_connect, \
> > @@ -376,27 +376,27 @@ pvconn_pstream_wait(struct pvconn *pvconn)
> > vconn_stream_run, \
> > vconn_stream_run_wait, \
> > vconn_stream_wait, \
> > - };
> > + }
> >
> > -#define DEFINE_PVCONN_STREAM_CLASS(NAME) \
> > - struct pvconn_class NAME##_pvconn_class = { \
> > - #NAME, \
> > +#define PSTREAM_INIT(NAME) \
> > + { \
> > + NAME, \
> > pvconn_pstream_listen, \
> > pvconn_pstream_close, \
> > pvconn_pstream_accept, \
> > pvconn_pstream_wait \
> > - };
> > + }
> >
> > -static DEFINE_VCONN_STREAM_CLASS(stream);
> > -static DEFINE_PVCONN_STREAM_CLASS(pstream);
> > +static struct vconn_class stream_vconn_class = STREAM_INIT("stream");
> > +static struct pvconn_class pstream_pvconn_class = PSTREAM_INIT("pstream");
> >
> > -DEFINE_VCONN_STREAM_CLASS(tcp);
> > -DEFINE_PVCONN_STREAM_CLASS(ptcp);
> > +struct vconn_class tcp_vconn_class = STREAM_INIT("tcp");
> > +struct pvconn_class ptcp_pvconn_class = PSTREAM_INIT("ptcp");
> >
> > -DEFINE_VCONN_STREAM_CLASS(unix);
> > -DEFINE_PVCONN_STREAM_CLASS(punix);
> > +struct vconn_class unix_vconn_class = STREAM_INIT("unix");
> > +struct pvconn_class punix_pvconn_class = PSTREAM_INIT("punix");
> >
> > #ifdef HAVE_OPENSSL
> > -DEFINE_VCONN_STREAM_CLASS(ssl);
> > -DEFINE_PVCONN_STREAM_CLASS(pssl);
> > +struct vconn_class ssl_vconn_class = STREAM_INIT("ssl");
> > +struct pvconn_class pssl_pvconn_class = PSTREAM_INIT("pssl");
> > #endif
> > --
> > 1.7.1
> >
> >
> > _______________________________________________
> > dev mailing list
> > dev at openvswitch.org
> > http://openvswitch.org/mailman/listinfo/dev_openvswitch.org
> >
More information about the dev
mailing list