On Mon, May 25, 2015 at 9:17 AM, Julia Lawall <[email protected]> wrote:
> On Mon, 25 May 2015, Fabio Estevam wrote:
>
>> The commit log should better describe the action that the patch will do.
>>
>> In this case we are 'adding' the pci_free_consistent.
>
> Not sure to follow the reasoning.  The semantic patch doesn't add
> anything.

Right, the semantich patch finds the missing pci_free_consistent, but
the generated patch is not finding pci_free_consistent. It is actually
adding the missing pci_free_consistent instead.

So in the commit log of the generated patch, we should better use
'add' instead of 'find'.

Regards,

Fabio Estevam
_______________________________________________
Cocci mailing list
[email protected]
https://systeme.lip6.fr/mailman/listinfo/cocci

Reply via email to