Changed ThreadFactory for executors (name threads)

Author: nostra13

library/src/com/nostra13/universalimageloader/core/DefaultConfigurationFactory.java

@@ -16,6 +16,8 @@
 package com.nostra13.universalimageloader.core;
 
 import java.io.File;
+import java.util.concurrent.ThreadFactory;
+import java.util.concurrent.atomic.AtomicInteger;
 
 import android.content.Context;
 import android.graphics.Bitmap;
@@ -91,4 +93,32 @@ public static ImageDownloader createImageDownloader(Context context) {
 	public static BitmapDisplayer createBitmapDisplayer() {
 		return new SimpleBitmapDisplayer();
 	}
+
+	public static ThreadFactory createThreadFactory(int threadPriority) {
+		return new DefaultThreadFactory(threadPriority);
+	}
+
+	private static class DefaultThreadFactory implements ThreadFactory {
+
+		private static final AtomicInteger poolNumber = new AtomicInteger(1);
+
+		private final ThreadGroup group;
+		private final AtomicInteger threadNumber = new AtomicInteger(1);
+		private final String namePrefix;
+		private final int threadPriority;
+
+		DefaultThreadFactory(int threadPriority) {
+			this.threadPriority = threadPriority;
+			SecurityManager s = System.getSecurityManager();
+			group = (s != null) ? s.getThreadGroup() : Thread.currentThread().getThreadGroup();
+			namePrefix = "pool-" + poolNumber.getAndIncrement() + "-thread-";
+		}
+
+		public Thread newThread(Runnable r) {
+			Thread t = new Thread(group, r, namePrefix + threadNumber.getAndIncrement(), 0);
+			if (t.isDaemon()) t.setDaemon(false);
+			t.setPriority(threadPriority);
+			return t;
+		}
+	}
 }

library/src/com/nostra13/universalimageloader/core/ImageLoaderConfiguration.java

@@ -15,8 +15,6 @@
  *******************************************************************************/
 package com.nostra13.universalimageloader.core;
 
-import java.util.concurrent.ThreadFactory;
-
 import android.content.Context;
 import android.graphics.Bitmap;
 import android.graphics.Bitmap.CompressFormat;
@@ -28,8 +26,8 @@
 import com.nostra13.universalimageloader.core.assist.FailReason;
 import com.nostra13.universalimageloader.core.assist.ImageLoadingListener;
 import com.nostra13.universalimageloader.core.assist.QueueProcessingType;
-import com.nostra13.universalimageloader.core.download.NetworkDeniedImageDownloader;
 import com.nostra13.universalimageloader.core.download.ImageDownloader;
+import com.nostra13.universalimageloader.core.download.NetworkDeniedImageDownloader;
 import com.nostra13.universalimageloader.core.download.SlowNetworkImageDownloader;
 import com.nostra13.universalimageloader.utils.L;
 
@@ -64,7 +62,7 @@ public final class ImageLoaderConfiguration {
 	final DiscCacheAware discCache;
 	final ImageDownloader downloader;
 	final DisplayImageOptions defaultDisplayImageOptions;
-	final ThreadFactory displayImageThreadFactory;
+	final int threadPriority;
 	final boolean loggingEnabled;
 
 	final DiscCacheAware reserveDiscCache;
@@ -87,14 +85,7 @@ private ImageLoaderConfiguration(final Builder builder) {
 		loggingEnabled = builder.loggingEnabled;
 		downloader = builder.downloader;
 		tasksProcessingType = builder.tasksProcessingType;
-		displayImageThreadFactory = new ThreadFactory() {
-			@Override
-			public Thread newThread(Runnable r) {
-				Thread t = new Thread(r);
-				t.setPriority(builder.threadPriority);
-				return t;
-			}
-		};
+		threadPriority = builder.threadPriority;
 
 		networkDeniedDownloader = new NetworkDeniedImageDownloader(downloader);
 		slowNetworkDownloader = new SlowNetworkImageDownloader(downloader);

library/src/com/nostra13/universalimageloader/core/ImageLoaderEngine.java

@@ -100,7 +100,7 @@ private ExecutorService createTaskExecutor() {
 		boolean lifo = configuration.tasksProcessingType == QueueProcessingType.LIFO;
 		BlockingQueue<Runnable> taskQueue = lifo ? new LIFOLinkedBlockingDeque<Runnable>() : new LinkedBlockingQueue<Runnable>();
 		return new ThreadPoolExecutor(configuration.threadPoolSize, configuration.threadPoolSize, 0L, TimeUnit.MILLISECONDS, taskQueue,
-				configuration.displayImageThreadFactory);
+				DefaultConfigurationFactory.createThreadFactory(configuration.threadPriority));
 	}
 
 	/** Returns URI of image which is loading at this moment into passed {@link ImageView} */