All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Ville Syrjälä" <ville.syrjala@linux.intel.com>
To: Julia Lawall <julia.lawall@inria.fr>
Cc: cocci@systeme.lip6.fr
Subject: Re: [Cocci] Inserting code after specific set of declarations?
Date: Thu, 20 Feb 2020 15:27:41 +0200	[thread overview]
Message-ID: <20200220132741.GX13686@intel.com> (raw)
In-Reply-To: <alpine.DEB.2.21.2002192335560.2565@hadrien>

On Wed, Feb 19, 2020 at 11:37:03PM +0100, Julia Lawall wrote:
> 
> 
> --- Please note the new email address ---
> 
> 
> On Wed, 19 Feb 2020, Ville Syrjälä wrote:
> 
> > Hi all,
> >
> > I was trying to do the following transformation:
> > @@
> > identifier M;
> > expression E;
> > @@
> > - struct foo M = E;
> > + struct foo M;
> > + copy_struct(&M, &E);
> >
> > but without inserting the function call in the middle of the
> > declarations and instead pushing it past them.
> >
> > This is one attempt:
> > @decl@
> > identifier M;
> > expression E;
> > @@
> > - struct foo M = E;
> > + struct foo M;
> >
> > @copy@
> > identifier decl.M;
> > expression decl.E;
> > declaration D;
> > statement S;
> > @@
> > struct foo M;
> > ...
> > D
> > + copy_struct(&M, &E);
> > S
> >
> > The copy rule fails to match anything when I have != 1
> > declarations after the struct. So the ... doesn't seem
> > to eat the extra declarations for some reason.
> 
> @copy@
> identifier decl.M;
> expression decl.E;
> declaration D;
> statement S;
> @@
> struct foo M;
> ... when any
> D
> + copy_struct(&M, &E);
> S
> 
> Otherwise, ... doesn't match code that includes what is before or after
> it, ie struct foo M or D.

This one still inserts the code after all declarations in nested
blocks. Also misses cases where there are no extra declarations.
So not quite what I need.

> 
> Aternatively:
> 
> @copy@
> identifier decl.M;
> expression decl.E;
> declaration D;
> statement S,S1;
> @@
> struct foo M;
> ... when != S1
> + copy_struct(&M, &E);
> S

This on the other hand seems to work very nicely. Thanks.

Earlier I tried to play around with some '... when !=' stuff,
but this double statement trick did not occur to me.

-- 
Ville Syrjälä
Intel
_______________________________________________
Cocci mailing list
Cocci@systeme.lip6.fr
https://systeme.lip6.fr/mailman/listinfo/cocci

      reply	other threads:[~2020-03-20  8:53 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-02-19 17:28 [Cocci] Inserting code after specific set of declarations? Ville Syrjälä
2020-02-19 22:37 ` Julia Lawall
2020-02-20 13:27   ` Ville Syrjälä [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20200220132741.GX13686@intel.com \
    --to=ville.syrjala@linux.intel.com \
    --cc=cocci@systeme.lip6.fr \
    --cc=julia.lawall@inria.fr \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.