Messages in this thread | | | Date | Fri, 5 Aug 2016 11:42:41 -0300 | From | Arnaldo Carvalho de Melo <> | Subject | Re: [RFC 0/4] tools lib traceevent: Install fixes |
| |
Em Fri, Aug 05, 2016 at 10:13:52AM -0400, Steven Rostedt escreveu: > On Tue, 2 Aug 2016 16:01:44 +0200 > Jiri Olsa <jolsa@redhat.com> wrote: > > > BTW, before we start making this ready for their own libraries, I'd > > > like to make some changes with the naming convention. Mainly with > > > event_format and format_field. > > > > > > Perhaps we should change them to pevent_event and pevent_field? > > > > right, it does not have any global prefix for public symbols > > > > I'd actualy expect something like 'traceevent_', > > but 'pevent' is ok as well I guess > > OK, what's the consensus here? Should we keep "pevent_" or switch to > the longer "traceevent_"?
traceevent_ matches libtraceevent, so I think would be more obvious the association of code with the library.
- Arnaldo
> I want to make the patch that gets this ready to ship! > > -- Steve
| |