Comment #10 on issue 16 by cy.delaunay: implicit importer is wrong: an
implicit variable should be attached to a method not a class
http://code.google.com/p/moose-technology/issues/detail?id=16
I did something to fix that. For now it is in my own repository, you can
load in the last 1.2 moose image by evaluating:
Gofer new
squeaksource: 'DelaunayTmpStuffs';
package: 'Moose-SmalltalkImporter';
version: 'Moose-SmalltalkImporter-cyrille_delaunay.103';
package: 'Moose-Tests-SmalltalkImporter-Core';
version: 'Moose-Tests-SmalltalkImporter-Core-cyrille_delaunay.13';
package: 'Moose-Tests-SmalltalkImporter-LAN';
version: 'Moose-Tests-SmalltalkImporter-LAN-cyrille_delaunay.5';
package: 'Famix-Extensions';
version: 'Famix-Extensions-cyrille_delaunay.150';
load.
What I mainly changed:
- added an instance variable in MooseImporter, to store them and be sure we
will not create two times the same implicit variable (not sure if my
solution is the best tn terms of performance, but it does not seem to take
too much to create a model).
- modify some tests that were expecting an implicitVariable to belong to a
class
- modify the code to compute an invocation's receiver's type.