Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1619362

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

From David Ahern <dsahern@gmail.com>
Newsgroups linux.kernel
Subject Re: [PATCH v2 5/5] perf tools: Refactor the code to strip command name with {l,r}trim()
Date 2017-04-08 20:50 +0200
Message-ID <tu3AR-v6-5@gated-at.bofh.it> (permalink)
References <ttMTn-6DM-9@gated-at.bofh.it> <ttMTo-6DM-21@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


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@gmail.com>
> Cc: Don Zickus <dzickus@redhat.com>
> Cc: Jiri Olsa <jolsa@kernel.org>
> Cc: Namhyung Kim <namhyung@kernel.org>
> Signed-off-by: Taeung Song <treeze.taeung@gmail.com>
> ---
>  tools/perf/util/event.c | 11 ++---------
>  1 file changed, 2 insertions(+), 9 deletions(-)
> 

Acked-by: David Ahern <dsahern@gmail.com>

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH v2 0/5] Refactoring with ltrim() and rtrim() Taeung Song <treeze.taeung@gmail.com> - 2017-04-08 03:00 +0200
  [PATCH v2 5/5] perf tools: Refactor the code to strip command name with {l,r}trim() Taeung Song <treeze.taeung@gmail.com> - 2017-04-08 03:00 +0200
    Re: [PATCH v2 5/5] perf tools: Refactor the code to strip command  name with {l,r}trim() David Ahern <dsahern@gmail.com> - 2017-04-08 20:50 +0200
      Re: [PATCH v2 5/5] perf tools: Refactor the code to strip command  name with {l,r}trim() Taeung Song <treeze.taeung@gmail.com> - 2017-04-09 07:40 +0200
  [PATCH v2 3/5] perf ui browser: Refactor the code to parse color configs with ltrim() Taeung Song <treeze.taeung@gmail.com> - 2017-04-08 03:00 +0200

csiph-web