* Hollis Blanchard ([email protected]) wrote: [...] > I believe the problem comes from -Werror=old-style-definition not liking > that empty "()", i.e. tracepoint_cb_qemu_tb_hash___flushall() { ... }. > The following patch seems to work for me; is there a reason it isn't > already written this way?
Good catch ! Fixed as master commit: commit 86637aa6452f66dee5f769461668c2ea059b3a30 Author: Mathieu Desnoyers <[email protected]> Date: Mon Jun 11 09:53:07 2012 -0400 Fix: tracepoint.h should not generate old-style definitions and merged into stable-2.0. Thanks! Mathieu > > diff --git a/include/lttng/tracepoint.h b/include/lttng/tracepoint.h > > index 4b773bb..60d8c73 100644 > > --- a/include/lttng/tracepoint.h > > +++ b/include/lttng/tracepoint.h > > @@ -88,7 +88,7 @@ extern "C" { > > #define _TP_EXDATA_VAR20(a,b,c,d,e,f,g,h,i,j,k,l,m,n,o,p,q,r,s,t) > __tp_data,b,d,f,h,j,l,n,p,r,t > > > > /* _TP_EXPROTO* extract tuples of type, var */ > > -#define _TP_EXPROTO0() > > +#define _TP_EXPROTO0() void > > #define _TP_EXPROTO2(a,b) a b > > #define _TP_EXPROTO4(a,b,c,d) a b,c d > > #define _TP_EXPROTO6(a,b,c,d,e,f) a b,c d,e f > > > If this is OK, I'm happy to resend as a proper patch. > > -- > Hollis Blanchard > Mentor Graphics, Embedded Systems Division > > > _______________________________________________ > lttng-dev mailing list > [email protected] > http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev -- Mathieu Desnoyers Operating System Efficiency R&D Consultant EfficiOS Inc. http://www.efficios.com _______________________________________________ lttng-dev mailing list [email protected] http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev
