Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Create a new error type for InterpreterResult #56

Open
boxbeam opened this issue Mar 21, 2023 · 3 comments
Open

Create a new error type for InterpreterResult #56

boxbeam opened this issue Mar 21, 2023 · 3 comments

Comments

@boxbeam
Copy link
Contributor

boxbeam commented Mar 21, 2023

No description provided.

@FuzzyNovaGoblin
Copy link
Contributor

I don't know what the plan for this was exactly. Is this a more immediate issue/something really short or is it something to be pushed back to after beta launch?

@boxbeam
Copy link
Contributor Author

boxbeam commented May 26, 2023

this was us wanting to unbox our errors in the interpreter because currently certain kinds are just held as a Box

@FuzzyNovaGoblin
Copy link
Contributor

Okay I don't think that is necessarily an issue with the boxes but just the ridiculous size of the errors since they are currently setup to just hold as much information as possible so we didn't have to rework things every time we were just trying to test things

I feel like this isn't one of the things we want to do before the beta launch, since the point of the current error type is to help when the language as a whole is constantly changing which I imagine will still happen during the beta

@FenderLang FenderLang deleted a comment from boxbeam May 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants