Re: [Xen-devel] [PATCH] xen/many: xfree() can tolerate NULL pointers

[Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>]

Ian Campbell writes ("Re: [PATCH] xen/many: xfree() can tolerate NULL 
pointers"):
> On Mon, 2015-01-19 at 10:42 +0000, Andrew Cooper wrote:
> > This was from some experimentation with semantic patches.  'spatch' can't
> > currently parse some of our macros (e.g. XEN_GUEST_HANDLE()), which cases it
> > to skip large numbers of functions in the codebase
> 
> How annoying! (and surprising)
> 
> Anyway, could you include the spatch in the commit log, for completeness
> and for future cargo culting ;-) (unless it's huge, I guess)

I think the spatch is the preferred form for modification of an
spatch-generated patch.  Therefore an spatch-generated patch should
not be accepted without the spatch.

(I would decline to add my own s-o-b to such a thing.)

Ian.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel