JC created TAVERNA-1019:
---------------------------
Summary: Condition that always returns true
Key: TAVERNA-1019
URL: https://issues.apache.org/jira/browse/TAVERNA-1019
Project: Apache Taverna
Issue Type: Bug
Components: Taverna Workbench
Reporter: JC
Priority: Trivial
Hi
In a recent snapshot from Github mirror for incubator-taverna-workbench, I've
found following suspicious code in
taverna-edits-impl/src/main/java/org/apache/taverna/workbench/edits/impl/menu/AbstractUndoAction.java
{code}
132 public void notify(Observable<EditManagerEvent> sender,
133 EditManagerEvent message) throws Exception {
134 if (!(message instanceof AbstractDataflowEditEvent))
135 return;
136 AbstractDataflowEditEvent dataflowEdit =
(AbstractDataflowEditEvent) message;
137 if
(dataflowEdit.getDataFlow().equals(dataflowEdit.getDataFlow()))
138 // It's an edit that could effect our
undoability
139 updateStatus();
140 }
{code}
In Line 137, equals compares the same expression. If this is intended, it might
not be an issue but wanted to report just in case. Thanks!
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)