On Fri, Jul 25, 2014 at 10:18:55AM +0900, Namhyung Kim wrote: > Set column width and do not change it if user gives -w/--column-widths > option. It'll truncate longer symbols than the width if exists. > > Signed-off-by: Namhyung Kim <namhy...@kernel.org> > ---
SNIP > > __HPP_COLOR_PERCENT_FN(overhead, period) > @@ -797,8 +798,11 @@ static int hist_browser__show_entry(struct hist_browser > *browser, > if (fmt->color) { > width -= fmt->color(fmt, &hpp, entry); > } else { > - width -= fmt->entry(fmt, &hpp, entry); > + int ret = fmt->entry(fmt, &hpp, entry); > + s[ret] = '\0'; > slsmg_printf("%s", s); from quick check it seems like all entry callback functions use some sort of snprint which should already handle the string length properly.. if not, we should fix that instead, no? 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/