On Mon, Jul 29, 2013 at 10:24:50AM -0700, Gurucharan Shetty wrote:
> On Mon, Jul 29, 2013 at 9:57 AM, Ben Pfaff <b...@nicira.com> wrote:
> 
> > In C, one can do preprocessor tricks by making a macro expansion include
> > the macro's own name.  We actually used this in the tree to automatically
> > provide function arguments, e.g.:
> >
> >     #define f(x) f(x, __FILE__, __LINE__)
> >     int f(int x, const char *file, int line);
> >
> > ...
> >
> >     f(1);    /* Expands to a call like f(1, __FILE__, __LINE__); */
> >
> > However it's somewhat confusing, so this commit stops using that trick.
> >
> > Reported-by: Ed Maste <ema...@freebsd.org>
> > Signed-off-by: Ben Pfaff <b...@nicira.com>
> >
> What is the commit on which this patch applies? I can't apply it on top of
> master.

There's a whole repo here:
https://github.com/blp/ovs-reviews/commits/cpp-trick
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to