ThreadService: only shutdown executor if it exists

ctrueden · ctrueden · commit f037ebf2bce4 · 2013-07-22T14:21:08.000-05:00
The previous behavior was rather silly: if the executor did not exist
yet, it was created on demand, only to be shutdown. What a waste.
diff --git a/src/main/java/org/scijava/thread/DefaultThreadService.java b/src/main/java/org/scijava/thread/DefaultThreadService.java
@@ -105,7 +105,7 @@ public void queue(final Runnable code) {
 
 	@Override
 	public void dispose() {
-		executor().shutdown();
+		if (executor != null) executor.shutdown();
 	}
 
 	// -- ThreadFactory methods --

Original file line number	Diff line number	Diff line change
`@@ -105,7 +105,7 @@ public void queue(final Runnable code) {`
`105`	`105`
`106`	`106`	`@Override`
`107`	`107`	`public void dispose() {`
`108`		`- executor().shutdown();`
	`108`	`+ if (executor != null) executor.shutdown();`
`109`	`109`	`}`
`110`	`110`
`111`	`111`	`// -- ThreadFactory methods --`