Hi guys
I loaded the latest version of moose OncifgurationOfMoose loadDefault. Then I loaded ConfigurationOfCAnalyze loadDefault then I opened a moose browser and tried to load C files but I got the following error ensureGlobalVariable: xmlDecl in: caUnit | variableName xmlVariableName famixVariable | self assert: [ xmlDecl name == #'decl' ]. and xmlDecl name -> 'decl' and not a symbol Stef _______________________________________________ Moose-dev mailing list [hidden email] https://www.iam.unibe.ch/mailman/listinfo/moose-dev |
On Apr 26, 2010, at 1:52 PM, stephane ducasse wrote: > Hi guys > > I loaded the latest version of moose OncifgurationOfMoose loadDefault. > Then I loaded ConfigurationOfCAnalyze loadDefault > > then I opened a moose browser and tried to load C files but I got the following error > > ensureGlobalVariable: xmlDecl in: caUnit > > | variableName xmlVariableName famixVariable | > self assert: [ xmlDecl name == #'decl' ]. > > and xmlDecl name -> 'decl' and not a symbol We fixed that with cyrille but now and some other protocol changes the are some expressions like expr / #'decl' should we turn them into expr / 'decl' ? > > Stef > _______________________________________________ > Moose-dev mailing list > [hidden email] > https://www.iam.unibe.ch/mailman/listinfo/moose-dev _______________________________________________ Moose-dev mailing list [hidden email] https://www.iam.unibe.ch/mailman/listinfo/moose-dev |
Hi Stef,
> We fixed that with cyrille but now > and some other protocol changes > the are some expressions like > > expr / #'decl' > > should we turn them into > expr / 'decl' > > > ? Yes, XMLParser changed recently. Strings have to be used instead of Symbols. For some reason, it is faster. But I run CAnalyzer tests before committing these changes in XMLParser. Cheers, Alexandre -- _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;: Alexandre Bergel http://www.bergel.eu ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;. _______________________________________________ Moose-dev mailing list [hidden email] https://www.iam.unibe.ch/mailman/listinfo/moose-dev |
On Apr 26, 2010, at 4:11 PM, Alexandre Bergel wrote: > Hi Stef, > >> We fixed that with cyrille but now >> and some other protocol changes >> the are some expressions like >> >> expr / #'decl' >> >> should we turn them into >> expr / 'decl' >> >> >> ? > > Yes, XMLParser changed recently. Strings have to be used instead of Symbols. For some reason, it is faster. it is faster because you do not have to intern them and make them unique. #symbol comparisons is faster than strings. but symbol creation is way slower. > But I run CAnalyzer tests before committing these changes in XMLParser. Well not the right ones :) Because lot of tests broken. > > Cheers, > Alexandre > -- > _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;: > Alexandre Bergel http://www.bergel.eu > ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;. > > > > > > _______________________________________________ > Moose-dev mailing list > [hidden email] > https://www.iam.unibe.ch/mailman/listinfo/moose-dev _______________________________________________ Moose-dev mailing list [hidden email] https://www.iam.unibe.ch/mailman/listinfo/moose-dev |
>> But I run CAnalyzer tests before committing these changes in
>> XMLParser. > > Well not the right ones :) > Because lot of tests broken. Humm... My mistake then. I should have made sure that CAnalyzer is not broken. Are you guys working on this? I think this is easy to fix. I can do it. Cheers, Alexandre > >> >> Cheers, >> Alexandre >> -- >> _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;: >> Alexandre Bergel http://www.bergel.eu >> ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;. >> >> >> >> >> >> _______________________________________________ >> Moose-dev mailing list >> [hidden email] >> https://www.iam.unibe.ch/mailman/listinfo/moose-dev > > > _______________________________________________ > Moose-dev mailing list > [hidden email] > https://www.iam.unibe.ch/mailman/listinfo/moose-dev -- _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;: Alexandre Bergel http://www.bergel.eu ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;. _______________________________________________ Moose-dev mailing list [hidden email] https://www.iam.unibe.ch/mailman/listinfo/moose-dev |
We fixed it.
Now for now we did not change expr / #dcl into expr / 'dcl' because it worked without. Stef On Apr 26, 2010, at 5:42 PM, Alexandre Bergel wrote: >>> But I run CAnalyzer tests before committing these changes in XMLParser. >> >> Well not the right ones :) >> Because lot of tests broken. > > Humm... My mistake then. I should have made sure that CAnalyzer is not broken. > Are you guys working on this? I think this is easy to fix. I can do it. > > Cheers, > Alexandre > >> >>> >>> Cheers, >>> Alexandre >>> -- >>> _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;: >>> Alexandre Bergel http://www.bergel.eu >>> ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;. >>> >>> >>> >>> >>> >>> _______________________________________________ >>> Moose-dev mailing list >>> [hidden email] >>> https://www.iam.unibe.ch/mailman/listinfo/moose-dev >> >> >> _______________________________________________ >> Moose-dev mailing list >> [hidden email] >> https://www.iam.unibe.ch/mailman/listinfo/moose-dev > > -- > _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;: > Alexandre Bergel http://www.bergel.eu > ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;. > > > > > > _______________________________________________ > Moose-dev mailing list > [hidden email] > https://www.iam.unibe.ch/mailman/listinfo/moose-dev _______________________________________________ Moose-dev mailing list [hidden email] https://www.iam.unibe.ch/mailman/listinfo/moose-dev |
Free forum by Nabble | Edit this page |