Problem
xmsgrid/ugrid/XmUGrid.cpp:704-707 — the failure branch of iMergeSegmentsToPoly calls XM_ASSERT(0) and clears a_polygon. XM_ASSERT is compiled out in non-debug builds (per xmscore/misc/XmError.h:60-70), leaving a_polygon.clear() as the sole failure signal. Combined with the silent-success bug tracked separately, release builds have no observable signal that polygon construction failed.
Suggested fix
Add an XM_LOG(xmlog::warning, ...) next to the assert so release builds at least surface the failure in logs, even before the public-API propagation issue is addressed.
Context
Surfaced during review of #193 (follow-up to #186). Reviewer flagged as out-of-scope for #193 and recommended filing separately. Related to the public-API silent-failure issue filed alongside this one.
Problem
xmsgrid/ugrid/XmUGrid.cpp:704-707— the failure branch ofiMergeSegmentsToPolycallsXM_ASSERT(0)and clearsa_polygon.XM_ASSERTis compiled out in non-debug builds (perxmscore/misc/XmError.h:60-70), leavinga_polygon.clear()as the sole failure signal. Combined with the silent-success bug tracked separately, release builds have no observable signal that polygon construction failed.Suggested fix
Add an
XM_LOG(xmlog::warning, ...)next to the assert so release builds at least surface the failure in logs, even before the public-API propagation issue is addressed.Context
Surfaced during review of #193 (follow-up to #186). Reviewer flagged as out-of-scope for #193 and recommended filing separately. Related to the public-API silent-failure issue filed alongside this one.