Surely it should be !oldValue.equals(e.getValue()) ?