Replace boost hash usage with TfHash in pxr/usd/pcp #2186
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description of Change(s)
Requires #2173
To remove the dependency of pxr/usd/pcp on boost's hashing functions
PcpInstanceKey
now relies onTfHash
's hashing of containers instead of its own iteration. To support this, the method_Arc::GetHash
has been replaced with aTfHashAppend
overloadPcpLayerStackIdentifier{,Str}
,PcpMapExpression
,PcpSite{,Str}
useTfHash::Combine
instead ofboost::hash_combine
PcpInstanceKey
andPcpMapFunction
PcpMapFunction
's hashing has been split off into an overload ofTfHashAppend
for its_Data
which isTfHash::Combine
d with the hash of its_offset
fieldFixes Issue(s)
-#2172 (additional PRs forthcoming)