Abstract vm.print away

[srijan-paul]

It'd be better to rename the print member variable to print_fn, and have vm.print call that, if it's not nullptr.
This way we can easily embed vyse in apps that do not have consoles enabled.

[svenkat19]

Could you specify under which directory I must do the changes?

[srijan-paul]

Could you specify under which directory I must do the changes?

Hi Shashank,
Currently when the vyse programmer calls the print function, on the C++ side of things, vm.print function is called along with the string that was passed to print. However, not all applications have a console to log stuff to, or perhaps sometimes it makes more sense to render to a screen instead (like browser's console.log). So what we ideally want is:

• Rename vm.print to vm.print_fn.
• Add a new vm.print method instead that first checks if vm.print_fn is nullptr, if so then it won't do anything and return.
• If the VM does have a print function to use (print_fn is not nullptr) , then it will call that.

I expect the changes will mainly go in the header and implementation file of the VM.
So lang/src/vm.cpp and lang/include/vyse/vm.hpp.

EDIT: If you'd like to contribute You'll have an easier time working on this once I merge the stdlib branch in.
So my suggestion would be create a branch off of the stdlib branch and then create a PR from your_branch to stdlib. The main branch is the stable version and is outdated by a couple of months.

[svenkat19]

Am I missing something , because I couldn't find the vm.print function in any of the mentioned files

[srijan-paul]

Am I missing something , because I couldn't find the vm.print function in any of the mentioned files

It's here, in the stdlib branch. The main branch is for stable builds.