maqetta/maqetta — comment on commit ccb68c4ee30257f03e30ed7b7ef8e0880f42e60d
File: maqetta.core.client/WebContent/davinci/ve/commands/RemoveCommand.js line 44
By JonFerraiolo:

I generally agree with @jhpedemonte. We should use the function parameter (and the closure) instead of the global variable. The ideal ultimate goal is that we remove all global variable references. (Of course, that ideal will likely take a long time to achieve.)

However, maybe there is a special situation here because the code above calls ```davinci.ve.states``` which might be different than ```States``` which actually means ```davinci.ve.States```, and then there is a difference between the singleton vs. the class. I haven't dug into this code to know whether there there is something special here that prevents the use of ```States```.