Messages in this thread | | | Subject | Re: [PATCH] tools lib traceevent: Modify header to work in C++ programs | From | Steven Rostedt <> | Date | Thu, 23 Aug 2012 11:17:09 -0400 |
| |
On Wed, 2012-08-22 at 15:47 +0900, Namhyung Kim wrote: > Hi, Steve > > On Tue, 21 Aug 2012 19:13:20 -0400, Steven Rostedt wrote: > > tools lib traceevent: Modify header to work in C++ programs > > > > Replace keyword "private" from event-parse.h to allow it to be > > used in C++ programs. > > > > Signed-off-by: Steven Rostedt <rostedt@goodmis.org> > > > > diff --git a/tools/lib/traceevent/event-parse.h b/tools/lib/traceevent/event-parse.h > > index 5772ad8..c28713b 100644 > > --- a/tools/lib/traceevent/event-parse.h > > +++ b/tools/lib/traceevent/event-parse.h > > @@ -49,7 +49,7 @@ struct pevent_record { > > int cpu; > > int ref_count; > > int locked; /* Do not free, even if ref_count is zero */ > > - void *private; > > + void *r_private; > > This r_ (and p_) prefix make it inconsistent with others. How about > simply using 'priv' instead?
Sure, sounds good.
I'll update and resend.
Thanks!
-- Steve
| |