commit 71483ae4917a2d0d4f768fcd70a5d34fb2f22a93
author Steffen Pingel <steffen.pingel@tasktop.com> Mon Sep 22 15:31:27 2014 +0200
committer Sam Davis <sam.davis@tasktop.com> Tue Oct 14 16:14:11 2014 -0400
tree 9ed069f529862f4e5847048271ebfeeb1b0f4a23
parent 1025e8adfa3b1f61dc1c07bc3ef0ca22140f046c

add @NonNull annotations to AttribueEditorFactory

Change-Id: I0871d7d13c1c5ce9cefe7e313e2ef1a262c94f68

org.eclipse.mylyn.tasks.ui/src/org/eclipse/mylyn/tasks/ui/editors/AttributeEditorFactory.java

diff --git a/org.eclipse.mylyn.tasks.ui/src/org/eclipse/mylyn/tasks/ui/editors/AttributeEditorFactory.java b/org.eclipse.mylyn.tasks.ui/src/org/eclipse/mylyn/tasks/ui/editors/AttributeEditorFactory.java
index 040ba78..404023e 100644
--- a/org.eclipse.mylyn.tasks.ui/src/org/eclipse/mylyn/tasks/ui/editors/AttributeEditorFactory.java
+++ b/org.eclipse.mylyn.tasks.ui/src/org/eclipse/mylyn/tasks/ui/editors/AttributeEditorFactory.java
@@ -12,6 +12,8 @@
 package org.eclipse.mylyn.tasks.ui.editors;
 
 import org.eclipse.core.runtime.Assert;
+import org.eclipse.jdt.annotation.NonNull;
+import org.eclipse.jdt.annotation.Nullable;
 import org.eclipse.mylyn.internal.tasks.ui.editors.BooleanAttributeEditor;
 import org.eclipse.mylyn.internal.tasks.ui.editors.DateAttributeEditor;
 import org.eclipse.mylyn.internal.tasks.ui.editors.DoubleAttributeEditor;
@@ -49,14 +51,15 @@
 
 	private AttributeEditorToolkit editorToolkit;
 
-	public AttributeEditorFactory(TaskDataModel model, TaskRepository taskRepository) {
+	public AttributeEditorFactory(@NonNull TaskDataModel model, @NonNull TaskRepository taskRepository) {
 		this(model, taskRepository, null);
 	}
 
 	/**
 	 * @since 3.1
 	 */
-	public AttributeEditorFactory(TaskDataModel model, TaskRepository taskRepository, IServiceLocator serviceLocator) {
+	public AttributeEditorFactory(@NonNull TaskDataModel model, @NonNull TaskRepository taskRepository,
+			@Nullable IServiceLocator serviceLocator) {
 		Assert.isNotNull(model);
 		Assert.isNotNull(taskRepository);
 		this.model = model;
@@ -67,6 +70,7 @@
 	/**
 	 * @since 3.1
 	 */
+	@Nullable
 	public AttributeEditorToolkit getEditorToolkit() {
 		return editorToolkit;
 	}
@@ -74,12 +78,14 @@
 	/**
 	 * @since 3.1
 	 */
-	public void setEditorToolkit(AttributeEditorToolkit editorToolkit) {
+	public void setEditorToolkit(@Nullable AttributeEditorToolkit editorToolkit) {
 		this.editorToolkit = editorToolkit;
 	}
 
-	public AbstractAttributeEditor createEditor(String type, TaskAttribute taskAttribute) {
+	@NonNull
+	public AbstractAttributeEditor createEditor(@NonNull String type, @NonNull TaskAttribute taskAttribute) {
 		Assert.isNotNull(type);
+		Assert.isNotNull(taskAttribute);
 
 		if (TaskAttribute.TYPE_BOOLEAN.equals(type)) {
 			return new BooleanAttributeEditor(model, taskAttribute);