Dev fix implicit conversion

[brtmax]

Description of Change(s)

Added explicit static_cast when converting to mt19937::result_type (uint_fast32_t) from:

• system_clock time count (64-bit integer)
• Large integer, 5109223000

Even though we're going from larger integers
to 32-bits, this is fine for random number generation seeding. The explicit cast makes this clearer in the code and removes the compiler warnings for implicit conversion.

The changes are in the test files: testHdSortedIds.cpp and testHdSortedIdsPerf.cpp, both of which still pass with the changes. I looked at the contribution and coding guidelines, if I overlooked something or there is a preferred way of implementing these changes, please let me know.

Link to proposal (if applicable)

Fixes Issue(s)

#3339

Checklist

[x] I have created this PR based on the dev branch

[x] I have followed the coding conventions

[ ] I have added unit tests that exercise this functionality (Reference:
testing guidelines)

[x] I have verified that all unit tests pass with the proposed changes

[x] I have submitted a signed Contributor License Agreement (Reference:
Contributor License Agreement instructions)

[jesschimein]

Filed as internal issue #USD-10663

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

[jesschimein]

/AzurePipelines run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).