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

Feature/bytecode types #43

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Feature/bytecode types #43

wants to merge 3 commits into from

Conversation

BitDragon256
Copy link
Collaborator

Resolves #27

This PR adds all native java types to the bytecode generator. Those are: byte, char, short, long, float, double. The arithmetic operations should all work, only the char type I couldn't test as it isn't yet supported by the semantic check.
I also needed to make some changes to the type handling in the background (in the stack and local variables), but those should not break any existing features.

@BitDragon256
Copy link
Collaborator Author

I just noticed that the tests are not running, will resolve this first

@Brofian
Copy link
Owner

Brofian commented Feb 3, 2025

I took the liberty to fix the bug that prevented chars from being recognized. The problem was, that the parser included the quotes of String- and char-literals in the value string. This caused all of them to be recognized as strings.
Also: you committed .class files inside the input directory. As I am sure, that was an accident, it removed them :)

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

Successfully merging this pull request may close these issues.

Full Type Support
2 participants