thorstenwagner / ij-ridgedetection

Detect ridges / lines with imagej
GNU General Public License v2.0
42 stars 27 forks source link

potential bad assignment in the "split the lines at the junction points" #39

Open lucalusn opened 5 years ago

lucalusn commented 5 years ago

In the code we have at least 100 value in the 'junc' variable, that are init with default values. These values are "really" filled starting from the last position, hence in some case could be that in this assignment https://github.com/thorstenwagner/ij-ridgedetection/blob/master/src/main/java/de/biomedical_imaging/ij/steger/Link.java#L1202 it assignes the default values instead of a real value

This situation could lead to unexpected behavior. In my case at the end of the code in the 'cont' vector I get duplicate values. Is it OK?