Fix warnings

Author: ctrueden

src/main/java/org/scijava/table/io/DefaultTableIOService.java

@@ -35,9 +35,7 @@
 
 import org.scijava.io.AbstractTypedIOService;
 import org.scijava.io.IOPlugin;
-import org.scijava.io.IOService;
 import org.scijava.io.location.Location;
-import org.scijava.plugin.Parameter;
 import org.scijava.plugin.Plugin;
 import org.scijava.service.Service;
 import org.scijava.table.Table;
@@ -84,7 +82,8 @@ public void save(Table<?, ?> table, String destination, TableIOOptions options)
 
 	@Override
 	public void save(Table<?, ?> table, Location destination, TableIOOptions options) throws IOException {
-		IOPlugin<Table> saver = ioService().getSaver(table, destination);
+		@SuppressWarnings("rawtypes")
+		final IOPlugin<Table> saver = ioService().getSaver(table, destination);
 		if (saver != null && TableIOPlugin.class.isAssignableFrom(saver.getClass())) {
 			((TableIOPlugin)saver).save(table, destination, options);
 		}

src/main/java/org/scijava/table/io/TableIOOptions.java

@@ -282,9 +282,10 @@ public Function<Object, String> formatter() {
 		 * @return the values of this column's options
 		 */
 		public ColumnTableIOOptions.Values column(int column) {
-			ColumnTableIOOptions columnOptions = getValueOrDefault(columnOptionsKey, new HashMap<Integer, ColumnTableIOOptions>()).get(column);
-			if(columnOptions == null) return null;
-			return columnOptions.values;
+			ColumnTableIOOptions colOpts = getValueOrDefault(columnOptionsKey,
+				new HashMap<Integer, ColumnTableIOOptions>()).get(column);
+			if (colOpts == null) return null;
+			return colOpts.values;
 		}
 	}
 }

src/main/java/org/scijava/table/io/TableIOPlugin.java

@@ -30,26 +30,35 @@
 
 package org.scijava.table.io;
 
+import java.io.IOException;
+
 import org.scijava.io.IOPlugin;
 import org.scijava.io.location.Location;
 import org.scijava.table.Table;
 
-import java.io.IOException;
-
 /**
  * Abstract plugin class for saving and loading tables
  *
  * @author Deborah Schmidt
  */
+@SuppressWarnings("rawtypes")
 public interface TableIOPlugin extends IOPlugin<Table> {
 
 	@Override
 	default Table<?, ?> open(Location source) throws IOException {
 		return open(source, new TableIOOptions());
 	}
 
-	/** Opens data from the given source. */
-	default Table<?, ?> open(final Location source, final TableIOOptions options) throws IOException {
+	/**
+	 * Opens data from the given source.
+	 *
+	 * @param source The data source to open as a table.
+	 * @param options The options to use when opening the data.
+	 * @throws IOException If something goes wrong opening the data source.
+	 */
+	default Table<?, ?> open(final Location source, final TableIOOptions options)
+		throws IOException
+	{
 		throw new UnsupportedOperationException();
 	}
 
@@ -58,8 +67,17 @@ default void save(Table data, Location destination) throws IOException {
 		save(data, destination, new TableIOOptions());
 	}
 
-	/** Saves the given data to the specified destination. */
-	default void save(final Table<?, ?> data, final Location destination, final TableIOOptions options) throws IOException {
+	/**
+	 * Saves the given data to the specified destination.
+	 *
+	 * @param data The table to save.
+	 * @param destination The destination where the table should be saved.
+	 * @param options The options to use when saving the data.
+	 * @throws IOException If something goes wrong opening the data source.
+	 */
+	default void save(final Table<?, ?> data, final Location destination,
+		final TableIOOptions options) throws IOException
+	{
 		throw new UnsupportedOperationException();
 	}
 

src/main/java/org/scijava/table/io/TableIOService.java

@@ -32,10 +32,8 @@
 
 import java.io.IOException;
 
-import org.scijava.io.IOService;
 import org.scijava.io.TypedIOService;
 import org.scijava.io.location.Location;
-import org.scijava.service.SciJavaService;
 import org.scijava.table.Table;
 
 public interface TableIOService extends TypedIOService<Table<?, ?>> {

src/test/java/org/scijava/table/io/TableIOServiceTest.java

@@ -122,10 +122,12 @@ public boolean supportsSave(Location loc) {
 		}
 
 		/**
-		 * This method creates a fake table for the purpose of testing the propagation of options.
-		 * It creates a row and a column with header names based on the {@param options}.
+		 * This method creates a fake table for the purpose of testing the
+		 * propagation of options. It creates a row and a column with header names
+		 * based on the {@code options}.
 		 */
 		@Override
+		@SuppressWarnings("unchecked")
 		public Table open(Location loc, TableIOOptions options) {
 			DefaultGenericTable table = new DefaultGenericTable();
 			if(options.values.readColumnHeaders()) {