-
Notifications
You must be signed in to change notification settings - Fork 17
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
Add a feature flag to automate Windows coloring #25
Comments
imo |
I think this is a rather special case. I don't want to care about the platform. Right now, I need to care about the platform. I cannot think of a single case where you would not want to write: if !Paint::enable_windows_ascii() {
Paint::disable()
}
What do you feel it does behind your back? |
|
These are fantastic points, @epage! I'll think about this a bit. Perhaps we simply want to try enabling coloring on Windows without disabling them if it doesn't work. Thus, a |
Tired of copy/pasting my herding of cats (all of the color use in my applications), I've done some research on where Rust is at wrt to color and have created a prototype with a PoC conversion of |
|
Basically, do the following automatically:
The implementation should just check if painting is disabled via the already relaxed atomic bool and otherwise check a second atomic bool, also relaxes, to see if we've tried to enable Windows coloring already.
The text was updated successfully, but these errors were encountered: