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

Fix: Build error with MSVC #3832

Merged
merged 1 commit into from
May 8, 2023

Conversation

LazyDodo
Copy link
Contributor

@LazyDodo LazyDodo commented May 8, 2023

Description

ssize_t is a posix type, msvc seemingly doesn't know about leading to a build error with MSVC 2019

usage elsewhere in this files used the py::ssize_t type, which seems to fix this

Checklist:

  • I have read the contribution guidelines.
  • If this is a small change, no CLA should be required
  • I have updated the documentation, if applicable. -> NA
  • I have ensured that the change is tested somewhere in the testsuite (N/A)
  • My code follows the prevailing code style of this project.

ssize_t is a posix type, msvc seemingly doesn't know about leading to a build error with MSVC 2019

usage elsewhere in this files used the `py::ssize_t` type, which seems to fix this
Copy link
Collaborator

@lgritz lgritz 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 for the fix

@lgritz lgritz merged commit 0a39c2c into AcademySoftwareFoundation:master May 8, 2023
19 of 20 checks passed
lgritz pushed a commit to lgritz/OpenImageIO that referenced this pull request May 9, 2023
`ssize_t` is a posix type, msvc seemingly doesn't know about leading to
a build error with MSVC 2019

usage elsewhere in this files used the `py::ssize_t` type, which seems
to fix this
lgritz pushed a commit to lgritz/OpenImageIO that referenced this pull request May 14, 2023
`ssize_t` is a posix type, msvc seemingly doesn't know about leading to
a build error with MSVC 2019

usage elsewhere in this files used the `py::ssize_t` type, which seems
to fix this
lgritz pushed a commit to lgritz/OpenImageIO that referenced this pull request May 15, 2023
`ssize_t` is a posix type, msvc seemingly doesn't know about leading to
a build error with MSVC 2019

usage elsewhere in this files used the `py::ssize_t` type, which seems
to fix this
lgritz pushed a commit to lgritz/OpenImageIO that referenced this pull request May 15, 2023
`ssize_t` is a posix type, msvc seemingly doesn't know about leading to
a build error with MSVC 2019

usage elsewhere in this files used the `py::ssize_t` type, which seems
to fix this
birsoyo pushed a commit to birsoyo/oiio that referenced this pull request Jun 8, 2023
`ssize_t` is a posix type, msvc seemingly doesn't know about leading to
a build error with MSVC 2019

usage elsewhere in this files used the `py::ssize_t` type, which seems
to fix this
birsoyo pushed a commit to birsoyo/oiio that referenced this pull request Jun 8, 2023
`ssize_t` is a posix type, msvc seemingly doesn't know about leading to
a build error with MSVC 2019

usage elsewhere in this files used the `py::ssize_t` type, which seems
to fix this
birsoyo pushed a commit to birsoyo/oiio that referenced this pull request Jun 8, 2023
`ssize_t` is a posix type, msvc seemingly doesn't know about leading to
a build error with MSVC 2019

usage elsewhere in this files used the `py::ssize_t` type, which seems
to fix this
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.

None yet

2 participants