search

CesiumGS / cesium-native

Apache License 2.0
414 stars 210 forks source link

Fix erroneous thread id assertion for classes derived from ReferenceCounted #821

Closed csciguy8 closed 6 months ago

csciguy8 commented 6 months ago

Noticed this when running Cesium for Unreal with the latest.

Fix bug where the client build system can define the order of members differently than cesium-native. Note the #ifndef NDEBUG statement.

The workaround is to just move the debug member to the end of the class, to preserve the non-debug order.

csciguy8 commented 6 months ago

Closing without merging. Will go another direction to solve this problem...