diff --git a/core/error_macros.h b/core/error_macros.h index 8fddac5216b..b20efd7de19 100644 --- a/core/error_macros.h +++ b/core/error_macros.h @@ -304,23 +304,6 @@ void _err_print_index_error(const char *p_function, const char *p_file, int p_li } else \ ((void)0) -/** - * Should assert only if making a build with dev asserts. - * This should be a 'free' check for program flow and should not be needed in any releases, - * only used in dev builds. - */ -// #define DEV_ASSERTS_ENABLED -#ifdef DEV_ASSERTS_ENABLED -#define DEV_ASSERT(m_cond) \ - if (unlikely(!(m_cond))) { \ - _err_print_error(FUNCTION_STR, __FILE__, __LINE__, "FATAL: DEV_ASSERT failed \"" _STR(m_cond) "\" is false."); \ - GENERATE_TRAP \ - } else \ - ((void)0) -#else -#define DEV_ASSERT(m_cond) -#endif - /** * If `m_cond` evaluates to `true`, crashes the engine immediately with a generic error message. * Only use this if there's no sensible fallback (i.e. the error is unrecoverable). @@ -535,3 +518,42 @@ void _err_print_index_error(const char *p_function, const char *p_file, int p_li ((void)0) #endif + +/** + * This should be a 'free' assert for program flow and should not be needed in any releases, + * only used in dev builds. + */ +#ifdef DEV_ENABLED +#define DEV_ASSERT(m_cond) \ + if (unlikely(!(m_cond))) { \ + _err_print_error(FUNCTION_STR, __FILE__, __LINE__, "FATAL: DEV_ASSERT failed \"" _STR(m_cond) "\" is false."); \ + GENERATE_TRAP \ + } else \ + ((void)0) +#else +#define DEV_ASSERT(m_cond) +#endif + +/** + * These should be 'free' checks for program flow and should not be needed in any releases, + * only used in dev builds. + */ +#ifdef DEV_ENABLED +#define DEV_CHECK(m_cond) \ + if (unlikely(!(m_cond))) { \ + ERR_PRINT("DEV_CHECK failed \"" _STR(m_cond) "\" is false."); \ + } else \ + ((void)0) +#else +#define DEV_CHECK(m_cond) +#endif + +#ifdef DEV_ENABLED +#define DEV_CHECK_ONCE(m_cond) \ + if (unlikely(!(m_cond))) { \ + ERR_PRINT_ONCE("DEV_CHECK_ONCE failed \"" _STR(m_cond) "\" is false."); \ + } else \ + ((void)0) +#else +#define DEV_CHECK_ONCE(m_cond) +#endif diff --git a/drivers/gles2/rasterizer_canvas_base_gles2.cpp b/drivers/gles2/rasterizer_canvas_base_gles2.cpp index 45baf4077b9..df4038d8352 100644 --- a/drivers/gles2/rasterizer_canvas_base_gles2.cpp +++ b/drivers/gles2/rasterizer_canvas_base_gles2.cpp @@ -641,7 +641,7 @@ void RasterizerCanvasBaseGLES2::_draw_gui_primitive(int p_points, const Vector2 stride += 1; } - RAST_DEV_DEBUG_ASSERT(p_points <= 4); + DEV_ASSERT(p_points <= 4); float buffer_data[(2 + 2 + 4 + 1) * 4]; for (int i = 0; i < p_points; i++) { diff --git a/drivers/gles2/rasterizer_canvas_gles2.cpp b/drivers/gles2/rasterizer_canvas_gles2.cpp index d7b0dd73bd7..f05c774739c 100644 --- a/drivers/gles2/rasterizer_canvas_gles2.cpp +++ b/drivers/gles2/rasterizer_canvas_gles2.cpp @@ -319,7 +319,7 @@ void RasterizerCanvasGLES2::render_batches(Item *p_current_clip, bool &r_reclip, default: { int end_command = batch.first_command + batch.num_commands; - RAST_DEV_DEBUG_ASSERT(batch.item); + DEV_ASSERT(batch.item); RasterizerCanvas::Item::Command *const *commands = batch.item->commands.ptr(); for (int i = batch.first_command; i < end_command; i++) { diff --git a/drivers/gles2/rasterizer_storage_gles2.h b/drivers/gles2/rasterizer_storage_gles2.h index f96dab26bf7..b1b4c0a8a62 100644 --- a/drivers/gles2/rasterizer_storage_gles2.h +++ b/drivers/gles2/rasterizer_storage_gles2.h @@ -1392,7 +1392,7 @@ inline void RasterizerStorageGLES2::buffer_orphan_and_upload(unsigned int p_buff } #endif } - RAST_DEV_DEBUG_ASSERT((p_offset + p_data_size) <= p_buffer_size); + DEV_ASSERT((p_offset + p_data_size) <= p_buffer_size); glBufferSubData(p_target, p_offset, p_data_size, p_data); } diff --git a/drivers/gles3/rasterizer_canvas_base_gles3.cpp b/drivers/gles3/rasterizer_canvas_base_gles3.cpp index bbda474e9fb..083b3980b03 100644 --- a/drivers/gles3/rasterizer_canvas_base_gles3.cpp +++ b/drivers/gles3/rasterizer_canvas_base_gles3.cpp @@ -554,7 +554,7 @@ void RasterizerCanvasBaseGLES3::_draw_gui_primitive(int p_points, const Vector2 stride += 1; } - RAST_DEV_DEBUG_ASSERT(p_points <= 4); + DEV_ASSERT(p_points <= 4); float b[(2 + 2 + 4 + 1) * 4]; for (int i = 0; i < p_points; i++) { diff --git a/drivers/gles3/rasterizer_canvas_gles3.cpp b/drivers/gles3/rasterizer_canvas_gles3.cpp index 701dc289457..ae7db0fd43b 100644 --- a/drivers/gles3/rasterizer_canvas_gles3.cpp +++ b/drivers/gles3/rasterizer_canvas_gles3.cpp @@ -500,7 +500,7 @@ void RasterizerCanvasGLES3::render_batches(Item *p_current_clip, bool &r_reclip, default: { int end_command = batch.first_command + batch.num_commands; - RAST_DEV_DEBUG_ASSERT(batch.item); + DEV_ASSERT(batch.item); RasterizerCanvas::Item::Command *const *commands = batch.item->commands.ptr(); for (int i = batch.first_command; i < end_command; i++) { diff --git a/drivers/gles3/rasterizer_storage_gles3.h b/drivers/gles3/rasterizer_storage_gles3.h index f44973dc7c1..6eb139aa269 100644 --- a/drivers/gles3/rasterizer_storage_gles3.h +++ b/drivers/gles3/rasterizer_storage_gles3.h @@ -1535,7 +1535,7 @@ inline void RasterizerStorageGLES3::buffer_orphan_and_upload(unsigned int p_buff } #endif } - RAST_DEV_DEBUG_ASSERT((p_offset + p_data_size) <= p_buffer_size); + DEV_ASSERT((p_offset + p_data_size) <= p_buffer_size); glBufferSubData(p_target, p_offset, p_data_size, p_data); } diff --git a/drivers/gles_common/rasterizer_canvas_batcher.h b/drivers/gles_common/rasterizer_canvas_batcher.h index 88e75802e49..f79bdaedecb 100644 --- a/drivers/gles_common/rasterizer_canvas_batcher.h +++ b/drivers/gles_common/rasterizer_canvas_batcher.h @@ -847,7 +847,8 @@ PREAMBLE(void)::_prefill_default_batch(FillState &r_fill_state, int p_command_nu // another default command, just add to the existing batch r_fill_state.curr_batch->num_commands++; - RAST_DEV_DEBUG_ASSERT(r_fill_state.curr_batch->num_commands <= p_command_num); + // Note this is getting hit, needs investigation as to whether this is a bug or a false flag + // DEV_CHECK_ONCE(r_fill_state.curr_batch->num_commands <= p_command_num); } else { #if defined(TOOLS_ENABLED) && defined(DEBUG_ENABLED) if (r_fill_state.transform_extra_command_number_p1 != p_command_num) { @@ -1744,7 +1745,7 @@ bool C_PREAMBLE::_prefill_polygon(RasterizerCanvas::Item::CommandPolygon *p_poly for (int n = 0; n < num_inds; n++) { int ind = p_poly->indices[n]; - RAST_DEV_DEBUG_ASSERT(ind < p_poly->points.size()); + DEV_CHECK_ONCE(ind < p_poly->points.size()); // recover at runtime from invalid polys (the editor may send invalid polys) if ((unsigned int)ind >= (unsigned int)num_verts) { @@ -1861,7 +1862,7 @@ PREAMBLE(bool)::_software_skin_poly(RasterizerCanvas::Item::CommandPolygon *p_po total_weight += weight; - RAST_DEBUG_ASSERT(bone_id < bone_count); + DEV_CHECK_ONCE(bone_id < bone_count); const Transform2D &bone_tr = bone_transforms[bone_id]; Vector2 pos = bone_tr.xform(src_pos_back_transformed); @@ -1904,7 +1905,7 @@ PREAMBLE(bool)::_software_skin_poly(RasterizerCanvas::Item::CommandPolygon *p_po for (int n = 0; n < num_inds; n++) { int ind = p_poly->indices[n]; - RAST_DEV_DEBUG_ASSERT(ind < num_verts); + DEV_CHECK_ONCE(ind < num_verts); // recover at runtime from invalid polys (the editor may send invalid polys) if ((unsigned int)ind >= (unsigned int)num_verts) { @@ -2730,7 +2731,7 @@ PREAMBLE(void)::join_sorted_items() { // but it is stupidly complex to calculate later, which would probably be slower. r->final_modulate = _render_item_state.final_modulate; } else { - RAST_DEBUG_ASSERT(_render_item_state.joined_item != nullptr); + DEV_ASSERT(_render_item_state.joined_item != nullptr); _render_item_state.joined_item->num_item_refs += 1; _render_item_state.joined_item->bounding_rect = _render_item_state.joined_item->bounding_rect.merge(ci->global_rect_cache); @@ -2939,7 +2940,7 @@ PREAMBLE(void)::_translate_batches_to_vertex_colored_FVF() { bdata.unit_vertices.prepare(sizeof(BatchVertexColored)); const BatchColor *source_vertex_colors = &bdata.vertex_colors[0]; - RAST_DEBUG_ASSERT(bdata.vertex_colors.size() == bdata.vertices.size()); + DEV_ASSERT(bdata.vertex_colors.size() == bdata.vertices.size()); int num_verts = bdata.vertices.size(); @@ -2979,8 +2980,8 @@ void C_PREAMBLE::_translate_batches_to_larger_FVF(uint32_t p_sequence_batch_type // the sizes should be equal, and allocations should never fail. Hence the use of debug // asserts to check program flow, these should not occur at runtime unless the allocation // code has been altered. - RAST_DEBUG_ASSERT(bdata.unit_vertices.max_size() == bdata.vertices.max_size()); - RAST_DEBUG_ASSERT(bdata.batches_temp.max_size() == bdata.batches.max_size()); + DEV_ASSERT(bdata.unit_vertices.max_size() == bdata.vertices.max_size()); + DEV_ASSERT(bdata.batches_temp.max_size() == bdata.batches.max_size()); Color curr_col(-1.0f, -1.0f, -1.0f, -1.0f); diff --git a/servers/visual/portals/portal_renderer.cpp b/servers/visual/portals/portal_renderer.cpp index 275d246221d..a766eef6ed7 100644 --- a/servers/visual/portals/portal_renderer.cpp +++ b/servers/visual/portals/portal_renderer.cpp @@ -64,10 +64,6 @@ OcclusionHandle PortalRenderer::instance_moving_create(VSInstance *p_instance, R } void PortalRenderer::instance_moving_update(OcclusionHandle p_handle, const AABB &p_aabb, bool p_force_reinsert) { - // we can ignore these, they are statics / dynamics, and don't need updating - // .. these should have been filtered out before calling the visual server... - DEV_ASSERT(!_occlusion_handle_is_in_room(p_handle)); - p_handle--; Moving &moving = _moving_pool[p_handle]; moving.exact_aabb = p_aabb; @@ -77,6 +73,10 @@ void PortalRenderer::instance_moving_update(OcclusionHandle p_handle, const AABB return; } + // we can ignore these, they are statics / dynamics, and don't need updating + // .. these should have been filtered out before calling the visual server... + DEV_CHECK_ONCE(!_occlusion_handle_is_in_room(p_handle)); + // quick reject for most roaming cases if (!p_force_reinsert && moving.expanded_aabb.encloses(p_aabb)) { return;