Attachments (0)
Change History (18)
comment:1 by , 4 years ago
comment:4 by , 4 years ago
Are you sure that this doesn't cause an endless loop when relation contains itself or when we have more complex loops like
a -> b, b-> c; c->a ?
nearestList.stream() .flatMap(o -> o.getReferrers().stream()) .filter(r -> r instanceof Relation && predicate.test(r)) .forEach(nearestList::add);
NavigatableComponent line 1610ff
comment:9 by , 4 years ago
Replying to GerdP:
Are you sure that this doesn't cause an endless loop when relation contains itself or when we have more complex loops like
a -> b, b-> c; c->a ?
No, thanks for spotting the error! Inlining the variable parentRelations
was faulty. I addressed this in r16444.
Edit: This error also has been reported by Java: https://josm.openstreetmap.de/jenkins/job/JOSM/6422/warnings9Result/module.1732282867/package.-515814635/
comment:16 by , 4 years ago
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
In 16436/josm: