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

Added a few POSIX definitions so it can compile with MinGW & PDCurses, and added command line parameters for colors. #3

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

wyatt8740
Copy link

Added a few POSIX definitions so it can compile with MinGW & PDCurses, and added command line parameters for colors.

…, added command line arguments for colors (see curses.h for the color definitions)
@erkyrath
Copy link
Owner

Thanks. I am in the middle of a game release so I haven't had time to look at this, but I'll get to it.

@wyatt8740
Copy link
Author

Thanks for looking!
It's 23 lines of additions not counting a few blank whitespace lines. I do think there may be a better way to implement the color arguments to avoid undefined numbers making the entire screen black. I just don't know how to do it.

@wyatt8740
Copy link
Author

I have looked back on this code with regret - it is BAD. I have another, better patch in the works.

(update - not had time to mess with it, and I think a couple #ifdef's might fix all the problems in my patch)

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