cpptool-develop

[Cpptool-develop] ExpressionMutator

[Sven R. <rei...@ma...>]

Baptiste,

I don't understand your code. Could you for example explain the parameters 
of the ExpressionMutator constructor?

Sven

-- 
Sven Reichard
Dept. of Math. Sci.
University of Delaware
rei...@ma...

Re: [Cpptool-develop] ExpressionMutator

[Baptiste L. <gai...@fr...>]

----- Original Message -----
From: "Sven Reichard" <rei...@ma...>
To: "CppTool Mailing List" <Cpp...@li...>
Sent: Friday, November 08, 2002 8:45 PM
Subject: [Cpptool-develop] ExpressionMutator


> Baptiste,
>
> I don't understand your code. Could you for example explain the parameters
> of the ExpressionMutator constructor?

Its called from a mutator such as DeclarationOrExpressionMutator (just like
the VariableDeclMutator), except that I moved the computation of the
start/end source range into the constructor since it makes no sense passing
anything else.

I added a test to help (and fixed a bug).

The parsing is done in two steps. Firstly, the expression is parsed and for
each 'token' in the expression, its range and token type are stored.
Secondly, the expression seen as a list of token is analysed and we search
the pattern that match an identifier (an identifier token which is neither
preceeded nor followed by a '.', '->' or '::').

I do like to think that this makes the code a lot clearer (at least much
more that the VariableDeclMutator).

A mutator (like DeclarationOrExpressionMutator ) still need to be added and
the ASTNodeTypes updated to help 'finding' identifier. Also, the variable
declaration statement node structure need to be changed to use a
variable-identifier instead of a variable-name.

Baptiste.

>
> Sven
>
> --
> Sven Reichard
> Dept. of Math. Sci.
> University of Delaware
> rei...@ma...

Re: [Cpptool-develop] ExpressionMutator

[Sven R. <rei...@ma...>]

Ok, I'll see if that helps. In the meantime I'm improving the parsing w/o 
using this mutator. I got one more of the rename temp tests to pass, 
although it broke something else... I'll fix that before committing.

Sven.


 On Fri, 8 Nov 2002, Baptiste Lepilleur wrote:
> Secondly, the expression seen as a list of token is analysed and we search
> the pattern that match an identifier (an identifier token which is neither
> preceeded nor followed by a '.', '->' or '::').

Why not followed by a '.' or '->'?

-- 
Sven Reichard
Dept. of Math. Sci.
University of Delaware
rei...@ma...

Re: [Cpptool-develop] ExpressionMutator

[Baptiste L. <gai...@fr...>]

----- Original Message -----
From: "Sven Reichard" <rei...@ma...>
To: "CppTool Mailing List" <Cpp...@li...>
Sent: Friday, November 08, 2002 10:31 PM
Subject: Re: [Cpptool-develop] ExpressionMutator


> Ok, I'll see if that helps. In the meantime I'm improving the parsing w/o
> using this mutator. I got one more of the rename temp tests to pass,
> although it broke something else... I'll fix that before committing.

OK.

> Sven.
>
>
>  On Fri, 8 Nov 2002, Baptiste Lepilleur wrote:
> > Secondly, the expression seen as a list of token is analysed and we
search
> > the pattern that match an identifier (an identifier token which is
neither
> > preceeded nor followed by a '.', '->' or '::').
>
> Why not followed by a '.' or '->'?

Did not though much when writting the code. I added more tests (and
refactored some). It should now behave as it is supposed to.

Baptiste.

>
> --
> Sven Reichard
> Dept. of Math. Sci.
> University of Delaware
> rei...@ma...