Remove unneeded case logic and improve javadoc

Author: ctrueden

src/main/java/org/scijava/ui/swing/script/InterpreterPane.java

@@ -71,22 +71,32 @@ public class InterpreterPane implements UIComponent<JComponent> {
 	private LogService log;
 
 	/**
-	 * Constructs an interpreter UI pane for a SciJava scripting REPL, with a given language preference.
+	 * Constructs an interpreter UI pane for a SciJava scripting REPL.
 	 *
-	 * @param context The SciJava application context to use
-	 * @param languagePreference The given language to use, or null to fall back to the default.
+	 * @param context The SciJava application context to use.
 	 */
-	public InterpreterPane(final Context context, final String languagePreference) {
+	public InterpreterPane(final Context context) {
+		this(context, null);
+	}
+
+	/**
+	 * Constructs an interpreter UI pane for a SciJava scripting REPL, with a
+	 * given language preference.
+	 *
+	 * @param context The SciJava application context to use.
+	 * @param languagePreference The given language to use, or null to fall back
+	 *          to the default.
+	 */
+	public InterpreterPane(final Context context,
+		final String languagePreference)
+	{
 		context.inject(this);
 		output = new OutputPane(log);
 		final JScrollPane outputScroll = new JScrollPane(output);
 		outputScroll.setPreferredSize(new Dimension(440, 400));
 
-		if(languagePreference == null) {
-			repl = new ScriptREPL(context, output.getOutputStream());
-		} else {
-			repl = new ScriptREPL(context, languagePreference, output.getOutputStream());
-		}
+		repl = new ScriptREPL(context, languagePreference, //
+			output.getOutputStream());
 		repl.initialize();
 
 		final Writer writer = output.getOutputWriter();
@@ -142,10 +152,6 @@ public void quit() {
 		mainPane.setDividerLocation(300);
 	}
 
-	public InterpreterPane(final Context context) {
-		this(context, null);
-	}
-
 	// -- InterpreterPane methods --
 
 	/** Gets the associated script REPL. */

src/main/java/org/scijava/ui/swing/script/InterpreterWindow.java

@@ -59,40 +59,42 @@ public class InterpreterWindow extends JFrame {
 	@Parameter
 	private LogService log;
 
-	/** Constructs the scripting interpreter window. */
-	public InterpreterWindow(final Context context, final String languagePreference) {
+	/**
+	 * Constructs the scripting interpreter window.
+	 *
+	 * @param context The SciJava application context to use.
+	 */
+	public InterpreterWindow(final Context context) {
+		this(context, null);
+	}
+
+	/**
+	 * Constructs the scripting interpreter window.
+	 *
+	 * @param context The SciJava application context to use.
+	 * @param languagePreference The given language to use, or null to fall back
+	 *          to the default.
+	 */
+	public InterpreterWindow(final Context context,
+		final String languagePreference)
+	{
 		super("Script Interpreter");
 		context.inject(this);
 
-		if(languagePreference == null) {
-			pane = new InterpreterPane(context) {
-				@Override
-				public void dispose() {
-					super.dispose();
-					InterpreterWindow.super.dispose();
-				}
-			};
-		} else {
-			pane = new InterpreterPane(context, languagePreference) {
-				@Override
-				public void dispose() {
-					super.dispose();
-					InterpreterWindow.super.dispose();
-				}
-			};
-		}
+		pane = new InterpreterPane(context, languagePreference) {
+			@Override
+			public void dispose() {
+				super.dispose();
+				InterpreterWindow.super.dispose();
+			}
+		};
 		setContentPane(pane.getComponent());
 
 		pack();
 
 		setDefaultCloseOperation(WindowConstants.DISPOSE_ON_CLOSE);
 	}
 
-	/** Construct the scripting interpreter with a given language preference. */
-	public InterpreterWindow(final Context context) {
-		this(context, null);
-	}
-
 	/** Gets the window's associated {@link ScriptREPL}. */
 	public ScriptREPL getREPL() {
 		return pane.getREPL();