On Fri, Sep 25, 2015 at 11:11:51AM +0000, He Kuang wrote:

SNIP

>  
> +static int config_term_tracepoint(struct perf_event_attr *attr,
> +                               struct parse_events_term *term,
> +                               struct parse_events_error *err)
> +{
> +     switch (term->type_term) {
> +     case PARSE_EVENTS__TERM_TYPE_CALLGRAPH:
> +     case PARSE_EVENTS__TERM_TYPE_STACKSIZE:
> +             return config_term_common(attr, term, err);
> +     default:
> +             if (err) {
> +                     err->idx = term->err_term;
> +                     err->str = strdup("unknown term");
> +                     err->help = strdup(
> +                             "valid terms: call-graph,stack-size\n");

please keep that on one line, we dont follow 80 chars rule strictly
if it results in ugly formatting like above

thanks,
jirka
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to