SerenityOS / jakt

The Jakt Programming Language
BSD 2-Clause "Simplified" License
2.77k stars 244 forks source link

Collection of codegen improvements #1560

Open cg-jl opened 2 months ago

cg-jl commented 2 months ago

This is a collection of small changes that have made no performance impact, but that I have made to achieve better readability on the generated code. I originally intended to make these into a proper PR, but uni started hitting me and it won't stop for at least one month more.

I don't intend to merge these changes as-is, but there are some things that we might want to pull from, or may serve as a list of low-hanging fruits for other devs to get into the weeds. I'll be separating the couple of changes that fix bugs I found in the codegen so they can be put into a different PR and properly merged.

stale[bot] commented 1 month ago

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days if no further activity occurs. Thank you for your contributions!

alimpfard commented 1 month ago

shoo bot, is draft.

stale[bot] commented 2 weeks ago

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days if no further activity occurs. Thank you for your contributions!

cg-jl commented 1 week ago

I started pulling the first thing that fixes binding common enum members and realized that we don't support it, so that one might have been me adding more code than necessary.