rokucommunity / brs

An interpreter for the BrightScript language that runs on non-Roku platforms.
MIT License
4 stars 2 forks source link

Fixed #38 - Improved context handling for Callables #40

Closed lvcabral closed 7 months ago

lvcabral commented 7 months ago

The original solution to identify the context (m object) for Callables was relying on re-evaluating the source on a dot chained call, that had performance issues, and caused the side effect of issue #9, the solution I implemented for it was partial, and did not solved the performance issue fully.

Now I saved a reference for the context for each callable, eliminating the need of re-evaluation, and fixed all side effects.

I added a couple of. new scenarios to the e2e test cases.