Commit cbfeb267 authored by John Kacur's avatar John Kacur Committed by Ingo Molnar

perf annotate: Add the cmp_null function and make use of it

This function exists in builtin-report.c but not in
builtin-annotate.c Functions that use cmp_null are shorter and
clearer.

Synchronizing functions between these two files will also make it
easier to potential share code in the future.
Signed-off-by: default avatarJohn Kacur <jkacur@redhat.com>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
LKML-Reference: <alpine.LFD.2.00.0909241754031.11383@localhost.localdomain>
Signed-off-by: default avatarIngo Molnar <mingo@elte.hu>
parent 725b1368
...@@ -82,6 +82,16 @@ struct sort_entry { ...@@ -82,6 +82,16 @@ struct sort_entry {
size_t (*print)(FILE *fp, struct hist_entry *); size_t (*print)(FILE *fp, struct hist_entry *);
}; };
static int64_t cmp_null(void *l, void *r)
{
if (!l && !r)
return 0;
else if (!l)
return -1;
else
return 1;
}
/* --sort pid */ /* --sort pid */
static int64_t static int64_t
...@@ -116,14 +126,8 @@ sort__comm_collapse(struct hist_entry *left, struct hist_entry *right) ...@@ -116,14 +126,8 @@ sort__comm_collapse(struct hist_entry *left, struct hist_entry *right)
char *comm_l = left->thread->comm; char *comm_l = left->thread->comm;
char *comm_r = right->thread->comm; char *comm_r = right->thread->comm;
if (!comm_l || !comm_r) { if (!comm_l || !comm_r)
if (!comm_l && !comm_r) return cmp_null(comm_l, comm_r);
return 0;
else if (!comm_l)
return -1;
else
return 1;
}
return strcmp(comm_l, comm_r); return strcmp(comm_l, comm_r);
} }
...@@ -149,14 +153,8 @@ sort__dso_cmp(struct hist_entry *left, struct hist_entry *right) ...@@ -149,14 +153,8 @@ sort__dso_cmp(struct hist_entry *left, struct hist_entry *right)
struct dso *dso_l = left->dso; struct dso *dso_l = left->dso;
struct dso *dso_r = right->dso; struct dso *dso_r = right->dso;
if (!dso_l || !dso_r) { if (!dso_l || !dso_r)
if (!dso_l && !dso_r) return cmp_null(dso_l, dso_r);
return 0;
else if (!dso_l)
return -1;
else
return 1;
}
return strcmp(dso_l->name, dso_r->name); return strcmp(dso_l->name, dso_r->name);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment