CesiumGS / cesium-unreal

Bringing the 3D geospatial ecosystem to Unreal Engine
https://cesium.com/platform/cesium-for-unreal/
Apache License 2.0
902 stars 287 forks source link

Cesium-native assertions no longer work with latest NDEBUG changes #1366

Open csciguy8 opened 5 months ago

csciguy8 commented 5 months ago

NDEBUG was defined for debug cesium-native builds as of this PR, https://github.com/CesiumGS/cesium-unreal/pull/1365

but it seems that asserts are no longer firing.

The intent of the PR was to solve any problems with mismatched headers between the cesium-native and plugin builds... but is it worth this tradeoff?