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

Use e_misc.h across overlays #1631

Merged
merged 15 commits into from
Sep 19, 2024
Merged

Conversation

bismurphy
Copy link
Collaborator

Good amount of deduplication here.

The only issue I ran into is that NZ0 has slightly different data for one array (it's missing two zeros at the end). I tried doing #undef STAGE #define STAGE STAGE_NZ0 in nz0.h and testing #if STAGE == STAGE_NZ0, but for some reason that was failing. I went with #define STAGE_IS_NZ0 and #if !defined(STAGE_IS_NZ0) for now; not sure what I was doing wrong.

Copy link
Owner

@Xeeynamo Xeeynamo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you so much for your hard work! The code base is looking better and better

@Xeeynamo Xeeynamo merged commit c20e982 into Xeeynamo:master Sep 19, 2024
15 checks passed
@bismurphy bismurphy deleted the cleanup_e_misc branch September 20, 2024 04:08
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.

2 participants