From: Baptiste L. <gai...@fr...> - 2002-11-16 11:26:23
|
I'm just done adding the ExpressionMutator to the MaxLODMutator. astdump now show identifiers in expressions. I also fixed a serious bug: the starting index of the identifier node was based on the expression starting index instead of the source (the test that was supposed to detect this was buggy). I fixed the RenameTempRefactoring that was based on this bug. Node indexes are always source based. I also fixed another bug revealed by using astdump: true, false and this were recognized as identifier with they are not. Also, the ExpressionMutator now creates localScopeIdentifier child nodes instead of variableIdentifier child nodes. Identifiers found in an expression could also be member functions, hence the renaming. Baptiste. ----- Original Message ----- From: "Baptiste Lepilleur" <gai...@fr...> To: "Sven Reichard" <rei...@ma...>; "CppTool Mailing List" <Cpp...@li...> Sent: Saturday, November 16, 2002 9:09 AM Subject: Re: [Cpptool-develop] update > I'll take the visitor. It's really just like the VariableDeclVisitor. > > Baptiste. > > ----- Original Message ----- > From: "Sven Reichard" <rei...@ma...> > To: "CppTool Mailing List" <Cpp...@li...> > Sent: Wednesday, November 13, 2002 8:18 PM > Subject: [Cpptool-develop] update > > > > I got rename temp to use the expr mutator. As a consequence, all existing > > tests pass now. However, the code is a bit of a mess; I'll try to clean it > > up. This should probably use a node/property visitor, but their mechanics > > isn't clear to me yet. Also, I suspect it won't work with some types of > > statements; tested so far are declarations, return, and expression > > statements. > > > > Cheers, > > Sven. > > > > -- > > Sven Reichard > > Dept. of Math. Sci. > > University of Delaware > > rei...@ma... > > > > > > > > ------------------------------------------------------- > > This sf.net email is sponsored by: Are you worried about > > your web server security? Click here for a FREE Thawte > > Apache SSL Guide and answer your Apache SSL security > > needs: http://www.gothawte.com/rd523.html > > _______________________________________________ > > Cpptool-develop mailing list > > Cpp...@li... > > https://lists.sourceforge.net/lists/listinfo/cpptool-develop > > > > > > > > > ------------------------------------------------------- > This sf.net email is sponsored by: To learn the basics of securing > your web site with SSL, click here to get a FREE TRIAL of a Thawte > Server Certificate: http://www.gothawte.com/rd524.html > _______________________________________________ > Cpptool-develop mailing list > Cpp...@li... > https://lists.sourceforge.net/lists/listinfo/cpptool-develop > > |