[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20171128133024.4f3b06e4@gandalf.local.home>
Date: Tue, 28 Nov 2017 13:30:24 -0500
From: Steven Rostedt <rostedt@...dmis.org>
To: Vladislav Valtchev <vladislav.valtchev@...il.com>
Cc: linux-kernel@...r.kernel.org, y.karadz@...il.com
Subject: Re: [PATCH 04/11] trace-cmd: Extract parse_record_options() from
trace_record()
On Tue, 28 Nov 2017 20:17:46 +0200
Vladislav Valtchev <vladislav.valtchev@...il.com> wrote:
> Since that code was profile-specific, I put it in trace_profile(),
> without 'if' statements, clearly.
This is all about balancing. You were able to remove one if statement,
but required two function calls by all others.
-- Steve
Powered by blists - more mailing lists