Skip to content

Clean Up - #59

Merged
tomdaley92 merged 9 commits into
mainfrom
feature/clean-up
Feb 22, 2026
Merged

Clean Up#59
tomdaley92 merged 9 commits into
mainfrom
feature/clean-up

Conversation

@tomdaley92

@tomdaley92 tomdaley92 commented Feb 18, 2026

Copy link
Copy Markdown
Member
  • Compile C and C++ object separately, linking at the end using C++ compiler
  • have each platform only implement open_fial_dialog.h, instead of providing another header
  • fix opcodes.h (remove opcode function declarations from chip8.h)
  • clean up makefiles

@tomdaley92 tomdaley92 self-assigned this Feb 18, 2026
* builds working on macos

* remove debug-vars target

* attempt linux build

* define M_PI manually

* let's try using gnu standard?

* add missing rule for linux/src dir

* does order matter?

* put linker flags at the end, where they were before

* make macos consistent

* convert long lines to multiline

* try windows builds here we go...
@tomdaley92
tomdaley92 marked this pull request as ready for review February 22, 2026 18:19
@tomdaley92
tomdaley92 merged commit af1c1b4 into main Feb 22, 2026
8 checks passed
@tomdaley92
tomdaley92 deleted the feature/clean-up branch February 22, 2026 18:20
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.

1 participant