summary |
shortlog |
log |
commit | commitdiff |
tree
raw |
patch |
inline | side by side (from parent 1:
bd10a06)
Only break if we actually found a tooltip, not unconditionally.
Cleanups.
Signed-off-by: Jens Axboe <axboe@kernel.dk>
static void __graph_value_drop(struct graph_label *l, struct graph_value *v)
{
static void __graph_value_drop(struct graph_label *l, struct graph_value *v)
{
+ flist_del_init(&v->list);
if (v->tooltip)
free(v->tooltip);
free(v->value);
if (v->tooltip)
free(v->tooltip);
free(v->value);
static void graph_value_drop(struct graph_label *l, struct graph_value *v)
{
static void graph_value_drop(struct graph_label *l, struct graph_value *v)
{
- struct flist_head *entry, *tmp;
-
/*
* Find head, the guy that's on the prio tree
*/
/*
* Find head, the guy that's on the prio tree
*/
- flist_for_each_safe(entry, tmp, &v->alias) {
+ while (!flist_empty(&v->alias)) {
- a = flist_entry(entry, struct graph_value, alias);
- flist_del(&a->alias);
+ a = flist_entry(v->alias.next, struct graph_value, alias);
+ flist_del_init(&a->alias);
__graph_value_drop(l, a);
}
__graph_value_drop(l, a);
}
/*
* If we got matches in one label, don't check others.
*/
/*
* If we got matches in one label, don't check others.
*/