Ted Unangst wrote:
> Michael McConville wrote:
> > There will probably be more similar patches to come if this is
> > acceptable. The legwork was done with the following Coccinelle script:
>
> I think it should be split up, but I'm happy to see diffs like this.
Is this small enough? It's just
On Mon, Sep 14, 2015 at 09:18:35AM -0400, Michael McConville wrote:
> Ted Unangst wrote:
> > Michael McConville wrote:
> > > There will probably be more similar patches to come if this is
> > > acceptable. The legwork was done with the following Coccinelle script:
> >
> > I think it should be
Michael McConville wrote:
> There will probably be more similar patches to come if this is
> acceptable. The legwork was done with the following Coccinelle script:
>
I think it should be split up, but I'm happy to see diffs like this.
There will probably be more similar patches to come if this is
acceptable. The legwork was done with the following Coccinelle script:
@@
expression E;
@@
- if(E) {
- free(E);
- }
+ free(E);
Index: bin/systrace/filter.c
===
Hi
I personally don't see a particular need for this, it's something that
can be tidied up as people make other changes in the area. Certainly you
are trying to touch a lot of different places at once here which is
usually not the best approach even for something minor. You'd probably
be better