You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@uima.apache.org by jo...@apache.org on 2018/12/04 15:04:02 UTC
svn commit: r1848160 - in /uima/uv3/uimaj-v3/trunk/uimaj-ep-cas-editor/src:
main/java/org/apache/uima/caseditor/editor/AnnotationStyle.java
test/java/org/apache/uima/caseditor/core/model/dotcorpus/AnnotationStyleTest.java
Author: joern
Date: Tue Dec 4 15:04:02 2018
New Revision: 1848160
URL: http://svn.apache.org/viewvc?rev=1848160&view=rev
Log:
UIMA-5925: Fix null error in AnnotationStyle.equals
Modified:
uima/uv3/uimaj-v3/trunk/uimaj-ep-cas-editor/src/main/java/org/apache/uima/caseditor/editor/AnnotationStyle.java
uima/uv3/uimaj-v3/trunk/uimaj-ep-cas-editor/src/test/java/org/apache/uima/caseditor/core/model/dotcorpus/AnnotationStyleTest.java
Modified: uima/uv3/uimaj-v3/trunk/uimaj-ep-cas-editor/src/main/java/org/apache/uima/caseditor/editor/AnnotationStyle.java
URL: http://svn.apache.org/viewvc/uima/uv3/uimaj-v3/trunk/uimaj-ep-cas-editor/src/main/java/org/apache/uima/caseditor/editor/AnnotationStyle.java?rev=1848160&r1=1848159&r2=1848160&view=diff
==============================================================================
--- uima/uv3/uimaj-v3/trunk/uimaj-ep-cas-editor/src/main/java/org/apache/uima/caseditor/editor/AnnotationStyle.java (original)
+++ uima/uv3/uimaj-v3/trunk/uimaj-ep-cas-editor/src/main/java/org/apache/uima/caseditor/editor/AnnotationStyle.java Tue Dec 4 15:04:02 2018
@@ -22,6 +22,7 @@ package org.apache.uima.caseditor.editor
import java.awt.Color;
import java.util.HashMap;
import java.util.Map;
+import java.util.Objects;
import org.apache.uima.caseditor.core.model.DefaultColors;
import org.eclipse.jface.preference.IPreferenceStore;
@@ -209,9 +210,8 @@ public final class AnnotationStyle {
} else if (object instanceof AnnotationStyle) {
AnnotationStyle style = (AnnotationStyle) object;
- boolean isConfigEqual = configuration == style.configuration ||
- (configuration != null ? false : configuration.equals(style.configuration));
-
+ boolean isConfigEqual = Objects.equals(configuration, style.configuration);
+
isEqual = annotation.equals(style.annotation) && this.style.equals(style.style)
&& color.equals(style.color) && layer == style.layer && isConfigEqual;
} else {
Modified: uima/uv3/uimaj-v3/trunk/uimaj-ep-cas-editor/src/test/java/org/apache/uima/caseditor/core/model/dotcorpus/AnnotationStyleTest.java
URL: http://svn.apache.org/viewvc/uima/uv3/uimaj-v3/trunk/uimaj-ep-cas-editor/src/test/java/org/apache/uima/caseditor/core/model/dotcorpus/AnnotationStyleTest.java?rev=1848160&r1=1848159&r2=1848160&view=diff
==============================================================================
--- uima/uv3/uimaj-v3/trunk/uimaj-ep-cas-editor/src/test/java/org/apache/uima/caseditor/core/model/dotcorpus/AnnotationStyleTest.java (original)
+++ uima/uv3/uimaj-v3/trunk/uimaj-ep-cas-editor/src/test/java/org/apache/uima/caseditor/core/model/dotcorpus/AnnotationStyleTest.java Tue Dec 4 15:04:02 2018
@@ -19,13 +19,14 @@
package org.apache.uima.caseditor.core.model.dotcorpus;
-import static org.junit.Assert.assertEquals;
-
import java.awt.Color;
import org.apache.uima.caseditor.editor.AnnotationStyle;
import org.junit.Test;
+import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertNotEquals;
+
/**
* Unit test for the {@link AnnotationStyle} class.
*/
@@ -42,6 +43,11 @@ public class AnnotationStyleTest {
255, 255, 0), 0);
assertEquals(a, b);
+
+ AnnotationStyle c = new AnnotationStyle("testType", AnnotationStyle.Style.BRACKET, new Color(
+ 255, 255, 0), 0, "config");
+
+ assertNotEquals(a, c);
}
/**