Messages in this thread | | | Date | Tue, 2 Aug 2016 23:14:11 +0900 | From | Namhyung Kim <> | Subject | Re: [RFC 0/4] tools lib traceevent: Install fixes |
| |
On Tue, Aug 02, 2016 at 04:01:44PM +0200, Jiri Olsa wrote: > On Tue, Aug 02, 2016 at 09:48:23AM -0400, Steven Rostedt wrote: > > SNIP > > > > > > > > > > > > > Also I think it'd be better for libtraceevent has dependency to libdl > > > > explicitly so that we can get rid of -ldl at the end. > > > > > > agreed, I'll queue your patch if v2 is needed > > > > > > > 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
The 'pevent' prefix would be more consistent to similar APIs IMHO.
Maybe it's worth splitting public header and internal header files?
Thanks, Namhyung
| |