Cache getter property values on BScript class

[wardlem]

This prevents having to perform the calculation multiple times. This requires the use of Object.defineProperty.

This requires that a script's tokens are immutable. Consider enforcing this by freezing (a copy of the) the token array and making the Token class's instance properties read-only (unfortunately, there is not a neat syntax for this built into the language).

[JBaczuk]

Would this occur often, normally (having to recalculate multiple times)?

[wardlem]

Would this occur often, normally (having to recalculate multiple times)?

Maybe. I could see, for example, the scriptType getter property being accessed repeatedly for the same script.

[JBaczuk]

Ok can we file this for future optimizations?

[wardlem]

Ok can we file this for future optimizations?

Yes, that is fine.