Em Fri, Aug 22, 2014 at 10:02:36AM +0200, Ingo Molnar escreveu:
> > > +struct callchain_print_arg {
> > > + /* for hists browser */
> > > + unsigned short row;
> > > + off_t row_offset;
> > > + bool is_current_entry;
> > > +
> > > + /* for file dump */
> > > + FILE *fp;
> > > + int printed;
> > > +};
>
> Just a data type definition nitpicking pet peeve of mine, don't
> you guys too find this vertically aligned form infinitely more
> readable:
>
> struct callchain_print_arg {
> /* for hists browser */
> unsigned short row;
> off_t row_offset;
> bool is_current_entry;
>
> /* for file dump */
> FILE *fp;
> int printed;
> };
>
> especially when looking at it via email, without syntax
> highlighting?
Agreed, will make it so when applying it.
- Arnaldo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
