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

macOS improvements and general warnings fixed #39

Merged
merged 3 commits into from
Jun 22, 2019

Conversation

tempelmann
Copy link

The first commit fixes some warnings I get with gcc on macOS in several shared source files. I took care to not break this for Linux & Win but did test this. Let me know if it causes problems.

The other commits improve the mac specific code, and add an Xcode project and explanations for easier development and debugging on a Mac.

@asalamon74
Copy link
Owner

According to #38 it's not really working. What do you suggest @tempelmann ? Should I just reject this pull request, or this could maybe be helpful later and merging to the macosx branch is useful?

@tempelmann
Copy link
Author

My pull req still fixes some issues, so accepting it would still help others who might try to take this further.

@asalamon74 asalamon74 merged commit c196846 into asalamon74:macosx Jun 22, 2019
@asalamon74
Copy link
Owner

Thanks @tempelmann, merged into the macosx branch.

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