Skip to content

[utils] Add a util to help printing with colors. NFC #4665

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

Merged
merged 3 commits into from
Sep 7, 2016
Merged

[utils] Add a util to help printing with colors. NFC #4665

merged 3 commits into from
Sep 7, 2016

Conversation

nkcsgexi
Copy link
Contributor

@nkcsgexi nkcsgexi commented Sep 7, 2016

Incorporating @DougGregor and @akyrtzi 's suggestion, I have moved the utility to a more sharable place.

Resolves SR-NNNN.

@nkcsgexi
Copy link
Contributor Author

nkcsgexi commented Sep 7, 2016

@swift-ci please test and merge

//===----------------------------------------------------------------------===//

#ifndef SWIFT_COLOR_UTILS_H
#define SWIFT_COLOR_UTILS_H
Copy link
Contributor

Choose a reason for hiding this comment

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

The macro guards style for all swift/Basic headers is to prefix with SWIFT_BASIC_, and follow with no underscores, so change to SWIFT_BASIC_COLORUTILS_H.

@nkcsgexi
Copy link
Contributor Author

nkcsgexi commented Sep 7, 2016

@swift-ci smoke test

1 similar comment
@nkcsgexi
Copy link
Contributor Author

nkcsgexi commented Sep 7, 2016

@swift-ci smoke test

@nkcsgexi nkcsgexi merged commit a901fc4 into swiftlang:master Sep 7, 2016
kateinoigakukun added a commit that referenced this pull request Aug 31, 2022
[pull] swiftwasm from main
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