mbox series

[0/3] libtraceevent: Various API fixes

Message ID 20181123133946.16239-1-tstoyanov@vmware.com (mailing list archive)
Headers show
Series libtraceevent: Various API fixes | expand

Message

Tzvetomir Stoyanov Nov. 23, 2018, 1:39 p.m. UTC
This patch series contains few unrelated changes in 
the library APIs. It improves the consistency of the APIs
return logic and removes a duplicated API. 

Tzvetomir Stoyanov (3):
  tools/lib/traceevent: Remove tep_data_event_from_type() API
  tools/lib/traceevent: Changed return logic of
    tep_register_event_handler() API
  tools/lib/traceevent: Changed return logic of trace_seq_printf() and
    trace_seq_vprintf() APIs

 .../libtraceevent-reg_event_handler.txt       | 12 +++++++---
 .../libtraceevent-tseq_print.txt              |  9 ++++----
 .../Documentation/libtraceevent.txt           |  1 -
 tools/lib/traceevent/event-parse.c            | 22 +++++++------------
 tools/lib/traceevent/event-parse.h            |  6 ++++-
 tools/lib/traceevent/trace-seq.c              | 17 +++++++++-----
 6 files changed, 39 insertions(+), 28 deletions(-)

Comments

Steven Rostedt Nov. 26, 2018, 2:34 p.m. UTC | #1
On Fri, 23 Nov 2018 13:39:58 +0000
Tzvetomir Stoyanov <tstoyanov@vmware.com> wrote:

> This patch series contains few unrelated changes in 
> the library APIs. It improves the consistency of the APIs
> return logic and removes a duplicated API. 
> 
> Tzvetomir Stoyanov (3):
>   tools/lib/traceevent: Remove tep_data_event_from_type() API
>   tools/lib/traceevent: Changed return logic of
>     tep_register_event_handler() API
>   tools/lib/traceevent: Changed return logic of trace_seq_printf() and
>     trace_seq_vprintf() APIs
> 
>  .../libtraceevent-reg_event_handler.txt       | 12 +++++++---
>  .../libtraceevent-tseq_print.txt              |  9 ++++----
>  .../Documentation/libtraceevent.txt           |  1 -
>  tools/lib/traceevent/event-parse.c            | 22 +++++++------------
>  tools/lib/traceevent/event-parse.h            |  6 ++++-
>  tools/lib/traceevent/trace-seq.c              | 17 +++++++++-----
>  6 files changed, 39 insertions(+), 28 deletions(-)
> 

This doesn't appear to apply on any tree.

Note, if you send a patch series, it must apply on the tree you are
sending it to. It should not depend on any patch that is pending and
has not yet been applied. If it does, then that pending patch needs to
be added to the series as well.

-- Steve