On Mon, 11 Aug 2014, SF Markus Elfring wrote:
> > @r depends on context || report || org @ > > @@ -37,7 +35,8 @@ position p; > > @@ > > > > * if (E) > > -* > > \(kfree@p\|debugfs_remove@p\|debugfs_remove_recursive@p\|usb_free_urb\)(E); > > +* > > \(kfree@p\|debugfs_remove@p\|debugfs_remove_recursive@p\|usb_free_urb@p\| > > +* of_node_put@p\)(E); > > Why do you want to limit this semantic patch pattern to such a small list of > function names? Because these are the ones that kernel developers have cited as interesting for this kind of transformation. > Would you like to consider also the hundreds of other functions which perform > input parameter validation? Personally, no. I think I have explained why several times already. julia > Are you interested to make the checked list more complete with further > (eventually automatic?) extensions? > > Regards, > Markus > -- > To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in > the body of a message to [email protected] > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
