410655: add @NonNull annotations to AbstractRepositorySettingsPage

Change-Id: I8f167f7e10cbe903a9a62f5e1e7273031effbdea
Task-Url: https://bugs.eclipse.org/bugs/show_bug.cgi?id=410655

2 files changed, 8 insertions, 7 deletions

diff --git a/org.eclipse.mylyn.tasks.ui/build.properties b/org.eclipse.mylyn.tasks.ui/build.properties
index 6472be2f2..0cc50f7b3 100644
--- a/org.eclipse.mylyn.tasks.ui/build.properties
+++ b/org.eclipse.mylyn.tasks.ui/build.properties
@@ -16,4 +16,5 @@ src.includes = about.html,\
                schema/
 source.. = src/
 
-jars.extra.classpath = platform:/plugin/org.eclipse.ui.navigator,platform:/plugin/org.eclipse.mylyn.discovery.ui
+additional.bundles = org.eclipse.jdt.annotation
+jars.extra.classpath = platform:/plugin/org.eclipse.ui.navigator,platform:/plugin/org.eclipse.mylyn.discovery.ui,platform:/plugin/org.eclipse.jdt.annotation
diff --git a/org.eclipse.mylyn.tasks.ui/src/org/eclipse/mylyn/tasks/ui/wizards/AbstractRepositorySettingsPage.java b/org.eclipse.mylyn.tasks.ui/src/org/eclipse/mylyn/tasks/ui/wizards/AbstractRepositorySettingsPage.java
index df1fd6776..a8618bde0 100644
--- a/org.eclipse.mylyn.tasks.ui/src/org/eclipse/mylyn/tasks/ui/wizards/AbstractRepositorySettingsPage.java
+++ b/org.eclipse.mylyn.tasks.ui/src/org/eclipse/mylyn/tasks/ui/wizards/AbstractRepositorySettingsPage.java
@@ -27,6 +27,7 @@ import org.eclipse.core.runtime.IProgressMonitor;
 import org.eclipse.core.runtime.IStatus;
 import org.eclipse.core.runtime.OperationCanceledException;
 import org.eclipse.core.runtime.Status;
+import org.eclipse.jdt.annotation.NonNull;
 import org.eclipse.jface.dialogs.Dialog;
 import org.eclipse.jface.dialogs.IMessageProvider;
 import org.eclipse.jface.layout.GridDataFactory;
@@ -1335,7 +1336,7 @@ public abstract class AbstractRepositorySettingsPage extends AbstractTaskReposit
 	/**
 	 * @since 2.0
 	 */
-	protected abstract void createAdditionalControls(Composite parent);
+	protected abstract void createAdditionalControls(@NonNull Composite parent);
 
 	/**
 	 * @since 2.0
@@ -1668,7 +1669,7 @@ public abstract class AbstractRepositorySettingsPage extends AbstractTaskReposit
 	 * @since 2.2
 	 */
 	@Override
-	public void applyTo(TaskRepository repository) {
+	public void applyTo(@NonNull TaskRepository repository) {
 		repository.setVersion(getVersion());
 		if (needsEncoding()) {
 			repository.setCharacterEncoding(getCharacterEncoding());
@@ -2014,7 +2015,7 @@ public abstract class AbstractRepositorySettingsPage extends AbstractTaskReposit
 	/**
 	 * @since 2.0
 	 */
-	protected abstract Validator getValidator(TaskRepository repository);
+	protected abstract Validator getValidator(@NonNull TaskRepository repository);
 
 	/**
 	 * Public for testing.
@@ -2025,7 +2026,7 @@ public abstract class AbstractRepositorySettingsPage extends AbstractTaskReposit
 
 		private IStatus status;
 
-		public abstract void run(IProgressMonitor monitor) throws CoreException;
+		public abstract void run(@NonNull IProgressMonitor monitor) throws CoreException;
 
 		public IStatus getStatus() {
 			return status;
@@ -2043,8 +2044,7 @@ public abstract class AbstractRepositorySettingsPage extends AbstractTaskReposit
 	 * @since 3.7
 	 * @see IAdaptable#getAdapter(Class)
 	 */
-	public Object getAdapter(@SuppressWarnings("rawtypes")
-	Class adapter) {
+	public Object getAdapter(@SuppressWarnings("rawtypes") Class adapter) {
 		if (adapter == IValidatable.class) {
 			return new IValidatable() {
 				public void validate() {