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

hd: add include of <chrono> in testHdSortedIds.cpp #3539

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

mattyjams
Copy link
Contributor

Description of Change(s)

The code makes use of std::chrono::system_clock, so this adds an include of the header in which that class is declared.

Fixes Issue(s)

This addresses a compilation failure on Windows when using Visual Studio 2022 17.13.1 (MSVC 14.43.34808):

-- Selecting Windows SDK version 10.0.22621.0 to target Windows 10.0.19045.
-- The C compiler identification is MSVC 19.43.34808.0
-- The CXX compiler identification is MSVC 19.43.34808.0
...
-- Check for working C compiler: C:/Program Files/Microsoft Visual Studio/2022/Professional/VC/Tools/MSVC/14.43.34808/bin/Hostx64/x64/cl.exe - skipped
...
-- Check for working CXX compiler: C:/Program Files/Microsoft Visual Studio/2022/Professional/VC/Tools/MSVC/14.43.34808/bin/Hostx64/x64/cl.exe - skipped

...

C:\Users\MattJohnson\Developer\OpenUSD\pxr\imaging\hd\testenv\testHdSortedIds.cpp(50,22): error C2039: 'system_clock': is not a member of 'std::chrono' [D:\OpenUSD_inst\build\OpenUSD\pxr\imaging\hd\testHdSortedIds.vcxproj]
      C:\Program Files\Microsoft Visual Studio\2022\Professional\VC\Tools\MSVC\14.43.34808\include\__msvc_chrono.hpp(286,11):
      see declaration of 'std::chrono'
  
C:\Users\MattJohnson\Developer\OpenUSD\pxr\imaging\hd\testenv\testHdSortedIds.cpp(50,22): error C3083: 'system_clock': the symbol to the left of a '::' must be a type [D:\OpenUSD_inst\build\OpenUSD\pxr\imaging\hd\testHdSortedIds.vcxproj]
C:\Users\MattJohnson\Developer\OpenUSD\pxr\imaging\hd\testenv\testHdSortedIds.cpp(50,36): error C2039: 'now': is not a member of 'std::chrono' [D:\OpenUSD_inst\build\OpenUSD\pxr\imaging\hd\testHdSortedIds.vcxproj]
      C:\Program Files\Microsoft Visual Studio\2022\Professional\VC\Tools\MSVC\14.43.34808\include\__msvc_chrono.hpp(286,11):
      see declaration of 'std::chrono'
  
C:\Users\MattJohnson\Developer\OpenUSD\pxr\imaging\hd\testenv\testHdSortedIds.cpp(50,36): error C3861: 'now': identifier not found [D:\OpenUSD_inst\build\OpenUSD\pxr\imaging\hd\testHdSortedIds.vcxproj]
C:\Users\MattJohnson\Developer\OpenUSD\pxr\imaging\hd\testenv\testHdSortedIds.cpp(49,18): error C2737: 'seed': const object must be initialized [D:\OpenUSD_inst\build\OpenUSD\pxr\imaging\hd\testHdSortedIds.vcxproj]

Checklist

The code makes use of `std::chrono::system_clock`, so this
adds an include of the header in which that class is declared.
@nvmkuruc nvmkuruc added the build Build-related issue/PR label Feb 21, 2025
@jesschimein
Copy link
Collaborator

Filed as internal issue #USD-10691

(This is an automated message. See here for more information.)

@jesschimein
Copy link
Collaborator

/AzurePipelines run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build Build-related issue/PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants