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

src: wrap MIN definition in infdef #1322

Closed
wants to merge 1 commit into from

Conversation

jbergstroem
Copy link
Member

Some platforms already define this; avoid redefining if that's the case. Found on OpenBSD 5.6.

/R=@bnoordhuis

Some platforms already define this; avoid redefining if that's
the case. Found on OpenBSD 5.6.
@mscdex mscdex added the c++ Issues and PRs that require attention from people who are familiar with C++. label Apr 1, 2015
@mscdex
Copy link
Contributor

mscdex commented Apr 1, 2015

LGTM

1 similar comment
@bnoordhuis
Copy link
Member

LGTM

jbergstroem added a commit that referenced this pull request Apr 2, 2015
Some platforms already define this; avoid redefining if that's
the case. Found on OpenBSD 5.6.

PR-URL: #1322
Reviewed-By: Brian White <mscdex@mscdex.net>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
@jbergstroem
Copy link
Member Author

Committed in f1e5a13. Thanks for the quick review.

@jbergstroem jbergstroem closed this Apr 2, 2015
@jbergstroem
Copy link
Member Author

typo in commit message. yay.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c++ Issues and PRs that require attention from people who are familiar with C++.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants