source-academy / frontend / 23734489899

41%

push

github

CSE Machine layout: Fixing referenced global functions throughout execution (#3695)

* Built-in/predeclared functions that are used in user code stays in global frame. Instead of referring to stash to check whether we need to include built-in/predeclared dynamically, we capture all of them that are needed, making them appear on global frames throughout.  (PS: this includes predelcared functions that are depended by other predeclared functions. For example: stream_map is dependent on pair, stream_tail,  is_null.

* improve cleaning up code logic and removed setDifference as no longer used

* yarn format

* sentry bot error fix

* include check for null rootnode to pass test

* minor changes to pass test

* format

* sentry bot fixes

* format

* minor changes

* format

* prevent user declarations from shadowing global built-ins via lexical scope analysis

* minor changes

---------

Co-authored-by: gigopogy <ethanlockzy@gmail.com>

2269 of 6457 branches covered (35.14%)

Branch coverage included in aggregate %.

46 of 64 new or added lines in 2 files covered. (71.88%)

5812 of 12840 relevant lines covered (45.26%)

29.01 hits per line