loadString merge=true create duplicate elements

[Adeel Asghar]

Run the attached script.

[Adrian Pop]

The problem in the script above does this:

setSourceFile(Test, "Test");

If you remove it, everything is fine.

The merge is implemented like:

1. get the old class and get its filename
2. remove *all* elements which have the same filename
3. merge with the contents of the new class

In this case we do:

1. get the old class and get its filename
   Test in this case was set via setSourceFile
2. remove *all* elements which have the same filename
   Removes nothing because all elements have filename
   <interactive> and not Test

So basically what happens is that we duplicate the class contents and then do union which keeps both parameter Integer N = 2; and Integer N = 2; because they are different.

I'll see how I can fix this.

[Adeel Asghar]

so you mean setSourceFile(Test, "Test"); doesn't set filename for all elements?

[Adrian Pop]

No. Just for that class, the elements inside the class are not affected.
I think it should behave like that anyway as if it would set all the elements then you might have issues with the packages loaded inside other packages.

[Martin Sjölund]

It should set it for all non-class elements (or the file merging should assume variables, etc are part of the class since they can't be part of another file).