Messages in this thread | | | Subject | Re: [PATCH 02/10] tracing/filter: Separate predicate init and filter addition | From | Steven Rostedt <> | Date | Wed, 10 Aug 2011 16:51:41 -0400 |
| |
On Thu, 2011-08-04 at 12:08 +0200, Jiri Olsa wrote: > Separating the filter_pred initialization from code > that adds the filter_pred to the filter. > > The create_pred returns initialized predicate, which > is added to the filter by filter_add_pred function. > > Keeping dry_run variable inside replace_preds function.
The only issue I have with this patch is the change log. It tells me what you are doing, but does not tell us why you are doing it.
I'm pretty clueless about reviewing this patch, as I don't know what the goal of the patch is. It's like a patch that says, increment i
+ i++;
-- Steve
> > Signed-off-by: Jiri Olsa <jolsa@redhat.com>
| |