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

Add includes for Windows #1827

Merged
merged 1 commit into from
Sep 19, 2024

Conversation

fnordware
Copy link
Contributor

Windows isn't happy unless I add these #includes. At least with Visual Studio 2017.

Signed-off-by: Brendan Bolles <[email protected]>
Copy link

linux-foundation-easycla bot commented Sep 18, 2024

CLA Signed


The committers listed above are authorized under a signed CLA.

Copy link
Member

@cary-ilm cary-ilm left a comment

Choose a reason for hiding this comment

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

LGTM, thanks

@cary-ilm cary-ilm merged commit 8f76f50 into AcademySoftwareFoundation:main Sep 19, 2024
34 checks passed
@fnordware fnordware deleted the win_includes branch September 19, 2024 16:52
asa pushed a commit to asa/openexr that referenced this pull request Sep 21, 2024
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