Adds reference to the instance name to symbol

[bigtimebuddy]

Changed

Modified the instance name setting to allow the name of the symbol to be set reflectively. This approach also accomodates other non alpha-numeric names (e.g., hyphens).

Old Approach

Old approach were instance names were set inline when instantiating the new instance.

var instance1 = this.example = new lib.Symbol_1()
    .setTransform(150.45, 85.7);

Minified

a=this.example=(new e.Symbol_1).t(150.45,85.7);

New Approach

Here the instance name is declared as a string and the minified code is slightly longer.

var instance1 = new lib.Symbol_1()
    .setTransform(150.45, 85.7);
this[instance1.name = "example"] = instance1;

Minified

a=(new e.Symbol_1).t(150.45,85.7);this[a.name="example"]=a

[andrewstart]

This looks good to me. As devil's advocate; in the interest of avoiding V8 hidden classes, would we instead want to have all Containers have a childrenByName map where we set this child, rather than the parent getting a new property for each child? I realize that it is more verbose and generally less friendly, but could have beneficial performance outcomes.

[bigtimebuddy]

@andrewstart I see what you're saying there. That most likely would be a breaking change, so probably a v2 approach to instance names. Another approach might be to have an instances property instead of a method. This would not create new properties on the class and would be a little easier to access instances, for instance clip.instances.example

[andrewstart]

I hadn't meant a method, but instances is a better name anyway.