On 12/21/2011 01:10 PM, Julia Lawall wrote: > On Wed, 21 Dec 2011, Lars-Peter Clausen wrote: > >> On 12/20/2011 11:24 PM, Julia Lawall wrote: >>> For this one, I made quite a number of changes, some of which required >>> fixing a bug in Coccinelle... I will try to release the bug fix >>> shortly. In general, though, I have added ... between the two tests. I >>> also got rid of the focus on ifs in the org and report rules. >>> >> >> Ok, thanks. Was the removing of the ifs in related to the Coccinelle >> bug? I >> always get an error if I start a match with a conditional. > > Not at all. The bug was that a patterm like: > > @depends on patch@ > expression x; > @@ > - IS_ERR(x) > + IS_ERR_OR_NULL(x) > || ... > - || x == NULL > > thought that it should be applied in two ways, giving an error about a > double transformation of the IS_ERR call. > > Could you reproduce the example that caused problems?
Hm, I can reproduce it anymore, even with my original patch, must have been caused by some other error I had in the semantic patch at that time. - Lars _______________________________________________ Cocci mailing list [email protected] http://lists.diku.dk/mailman/listinfo/cocci (Web access from inside DIKUs LAN only)
