Closed btwooton closed 6 years ago
Hey guys. Please try review this pull request soon. The changes to fix Symbol are kind of important and I'd like to be able to rebase the work in my current pull-request-VirtualMachine branch on those changes as I have begun to write unit tests and develop the functionality for the evaluate_assembly_instruction()
method.
The CallFrame class has been implemented. As we discussed yesterday (9/27/2017) this is simply a dummy class with only getter and setter methods for its member variables. Therefore, there are no unit tests for this class, but I did ensure compilation by including the CallFrame.hpp in a test file and building it. Please review the comments in the CallFrame.hpp file and let me know if you would like to see anything changed or reworded.