Skip to content

allow color API to enable and disable colors#2111

Merged
kripken merged 1 commit intoWebAssembly:masterfrom
tweag:may-16-provide-finer-grained-color-API
May 17, 2019
Merged

allow color API to enable and disable colors#2111
kripken merged 1 commit intoWebAssembly:masterfrom
tweag:may-16-provide-finer-grained-color-API

Conversation

@bollu
Copy link
Copy Markdown
Contributor

@bollu bollu commented May 16, 2019

This is useful for front-ends which wish to selectively enable or
disable coloring.

Also expose this API through the C API

Copy link
Copy Markdown
Member

@kripken kripken left a comment

Choose a reason for hiding this comment

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

Thanks for the PR!

Comment thread src/binaryen-c.cpp Outdated
return NULL;
}

void BinaryenSetColosrEnabled(int enabled) { Colors::setEnabled(enabled); }
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

typo in Colosr

Comment thread src/binaryen-c.h Outdated
Comment thread src/support/colors.h Outdated
@kripken
Copy link
Copy Markdown
Member

kripken commented May 16, 2019

Have you joined the w3c group (which we require from contributors)?

For a test, you can add a program in test/example/, like perhaps inside the kitchen sink example there which tests most of the C API. We can't actually check for colors working, but at least that the API exists I guess.

@bollu
Copy link
Copy Markdown
Contributor Author

bollu commented May 17, 2019

Yes, I just joined the WebAssembly community group: https://www.w3.org/community/webassembly/participants (Siddharth Bhat)

This is useful for front-ends which wish to selectively enable or
disable coloring.

Also expose these APIs from the C API.
@kripken
Copy link
Copy Markdown
Member

kripken commented May 17, 2019

Great, thanks!

@kripken kripken merged commit 1184678 into WebAssembly:master May 17, 2019
@TerrorJack TerrorJack deleted the may-16-provide-finer-grained-color-API branch July 20, 2019 13:06
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