Rename StderrLogService to DefaultLogService

This is a backwards incompatible change.

Author: ctrueden

pom.xml

@@ -10,7 +10,7 @@
 	</parent>
 
 	<artifactId>scijava-common</artifactId>
-	<version>2.39.1-SNAPSHOT</version>
+	<version>3.0.0-SNAPSHOT</version>
 
 	<name>SciJava Common</name>
 	<description>SciJava Common is a shared library for SciJava software. It provides a plugin framework, with an extensible mechanism for service discovery, backed by its own annotation processor, so that plugins can be loaded dynamically. It is used by both ImageJ and SCIFIO.</description>

src/main/java/org/scijava/log/DefaultLogService.java

@@ -0,0 +1,67 @@
+/*
+ * #%L
+ * SciJava Common shared library for SciJava software.
+ * %%
+ * Copyright (C) 2009 - 2015 Board of Regents of the University of
+ * Wisconsin-Madison, Broad Institute of MIT and Harvard, and Max Planck
+ * Institute of Molecular Cell Biology and Genetics.
+ * %%
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions are met:
+ * 
+ * 1. Redistributions of source code must retain the above copyright notice,
+ *    this list of conditions and the following disclaimer.
+ * 2. Redistributions in binary form must reproduce the above copyright notice,
+ *    this list of conditions and the following disclaimer in the documentation
+ *    and/or other materials provided with the distribution.
+ * 
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS"
+ * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
+ * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
+ * ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDERS OR CONTRIBUTORS BE
+ * LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
+ * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
+ * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
+ * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
+ * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
+ * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
+ * POSSIBILITY OF SUCH DAMAGE.
+ * #L%
+ */
+
+package org.scijava.log;
+
+import org.scijava.Priority;
+import org.scijava.plugin.Plugin;
+import org.scijava.service.Service;
+
+/**
+ * Implementation of {@link LogService} using the standard error stream.
+ * 
+ * @author Johannes Schindelin
+ * @author Curtis Rueden
+ */
+@Plugin(type = Service.class, priority = Priority.LOW_PRIORITY)
+public class DefaultLogService extends AbstractLogService {
+
+	/**
+	 * Prints a message to stderr.
+	 * 
+	 * @param message the message
+	 */
+	@Override
+	protected void log(final String message) {
+		System.err.println(message);
+	}
+
+	/**
+	 * Prints an exception to stderr.
+	 * 
+	 * @param t the exception
+	 */
+	@Override
+	protected void log(final Throwable t) {
+		t.printStackTrace();
+	}
+
+}

src/main/java/org/scijava/script/AbstractScriptEngine.java

@@ -39,8 +39,8 @@
 import javax.script.ScriptEngineFactory;
 import javax.script.ScriptException;
 
+import org.scijava.log.DefaultLogService;
 import org.scijava.log.LogService;
-import org.scijava.log.StderrLogService;
 
 /**
  * This class implements dummy versions for ScriptEngine's methods that are not
@@ -70,7 +70,7 @@ public abstract class AbstractScriptEngine implements ScriptEngine {
 
 	public synchronized LogService log() {
 		if (log == null) {
-			log = new StderrLogService();
+			log = new DefaultLogService();
 		}
 		return log;
 	}

src/main/java/org/scijava/service/ServiceHelper.java

@@ -43,8 +43,8 @@
 import org.scijava.Optional;
 import org.scijava.event.EventHandler;
 import org.scijava.event.EventService;
+import org.scijava.log.DefaultLogService;
 import org.scijava.log.LogService;
-import org.scijava.log.StderrLogService;
 import org.scijava.plugin.Parameter;
 import org.scijava.plugin.PluginInfo;
 import org.scijava.service.event.ServicesLoadedEvent;
@@ -114,7 +114,7 @@ public ServiceHelper(final Context context,
 	{
 		setContext(context);
 		log = context.getService(LogService.class);
-		if (log == null) log = new StderrLogService();
+		if (log == null) log = new DefaultLogService();
 		classPoolMap = new HashMap<Class<? extends Service>, Double>();
 		classPoolList = new ArrayList<Class<? extends Service>>();
 		findServiceClasses(classPoolMap, classPoolList);

src/test/java/org/scijava/ContextCreationTest.java

@@ -98,7 +98,7 @@ public void testFull() {
 				org.scijava.ui.dnd.DefaultDragAndDropService.class,
 				org.scijava.welcome.DefaultWelcomeService.class,
 				org.scijava.widget.DefaultWidgetService.class,
-				org.scijava.log.StderrLogService.class,
+				org.scijava.log.DefaultLogService.class,
 				org.scijava.platform.DefaultAppEventService.class };
 
 		final Context context = new Context();

src/test/java/org/scijava/log/LogServiceTest.java

@@ -44,7 +44,7 @@
 public class LogServiceTest {
 	@Test
 	public void testDefaultLevel() {
-		final LogService log = new StderrLogService();
+		final LogService log = new DefaultLogService();
 		int level = log.getLevel();
 		assertTrue("default level (" + level + ") is at least INFO(" + WARN + ")", level >= WARN);
 	}

src/test/java/org/scijava/service/ServiceIndexTest.java

@@ -40,7 +40,7 @@
 import org.junit.Test;
 import org.scijava.Context;
 import org.scijava.event.DefaultEventService;
-import org.scijava.log.StderrLogService;
+import org.scijava.log.DefaultLogService;
 import org.scijava.options.DefaultOptionsService;
 import org.scijava.options.OptionsService;
 import org.scijava.plugin.DefaultPluginService;
@@ -63,7 +63,7 @@ public void testGetAll() {
 		assertSame(DefaultEventService.class, all.get(0).getClass());
 		assertSame(DefaultPluginService.class, all.get(1).getClass());
 		assertSame(DefaultThreadService.class, all.get(2).getClass());
-		assertSame(StderrLogService.class, all.get(3).getClass());
+		assertSame(DefaultLogService.class, all.get(3).getClass());
 	}
 
 	/**