bound.next_edge: Not initialized before first call#69
bound.next_edge: Not initialized before first call#69flippmoke merged 1 commit intomapbox:masterfrom
Conversation
bound.hpp: Add next_edge initialization snap_rounding.hpp: In insert_local_minima_into_ABL_hot_pixel with current_edge also initialized next_edge
|
@DiabloRusso thank you for finding this. This helped me realized that I'd be wrongly assuming for quite some time now that the But I pushed a test branch enabling it on travis, where we do test with g++, and sure enough this bug is caught (since we turn warnings into errors): https://travis-ci.org/mapbox/wagyu/jobs/202717641#L348 |
|
Rebased against master, the |
bound.hpp: Add next_edge initialization
snap_rounding.hpp: In insert_local_minima_into_ABL_hot_pixel with
current_edge also initialized next_edge