Poly's moveHandle() always returns null

Author: MarcinOrlowskiCreated Sep 4, 2021Updated Sep 7, 2026
Labelspri -1investigating

In src/main/java/com/cburch/draw/shapes/Poly.java line 319, moveHandle() method reads:

java
  @Override
  public Handle moveHandle(HandleGesture gesture) {
    final var hs = getHandles(gesture);
    final var is = new Handle[hs.size()];
    Handle ret = null;
    var i = -1;
    for (final var h : hs) {
      i++;
      is[i] = h;
    }
    setHandles(is);
    return ret;
  }

my concern is return value as it is always null. And since this is consumed in other parts, I wonder if this is intentional but undocumented, or bug. Anyone knows more?

Source: logisim-evolution/logisim-evolution