Re: [PATCH v2 5/5] perf tools: Refactor the code to strip command name with {l,r}trim()

From: Taeung Song
Date: Sun Apr 09 2017 - 01:41:10 EST


Hi David,

On 04/09/2017 03:48 AM, David Ahern wrote:
On 4/7/17 8:52 PM, Taeung Song wrote:
After reading command name from /proc/<pid>/status,
use ltrim() and rtrim() to strip command name, not using
just while loop, isspace() and etc.

Cc: David Ahern <dsahern@xxxxxxxxx>
Cc: Don Zickus <dzickus@xxxxxxxxxx>
Cc: Jiri Olsa <jolsa@xxxxxxxxxx>
Cc: Namhyung Kim <namhyung@xxxxxxxxxx>
Signed-off-by: Taeung Song <treeze.taeung@xxxxxxxxx>
---
tools/perf/util/event.c | 11 ++---------
1 file changed, 2 insertions(+), 9 deletions(-)


Acked-by: David Ahern <dsahern@xxxxxxxxx>


Thank you!

- Taeung