Reuse BasicThreadFactory

Author: garydgregory

commons-vfs2/src/main/java/org/apache/commons/vfs2/impl/DefaultFileMonitor.java

@@ -20,8 +20,10 @@
 import java.util.HashMap;
 import java.util.Map;
 import java.util.Stack;
+import java.util.concurrent.ThreadFactory;
 import java.util.stream.Stream;
 
+import org.apache.commons.lang3.concurrent.BasicThreadFactory;
 import org.apache.commons.logging.Log;
 import org.apache.commons.logging.LogFactory;
 import org.apache.commons.vfs2.FileListener;
@@ -64,11 +66,11 @@
  *
  * <pre>
  * FileSystemManager fsManager = VFS.getManager();
- * FileObject listendir = fsManager.resolveFile("/home/username/monitored/");
+ * FileObject listenDir = fsManager.resolveFile("/home/username/monitored/");
  *
  * DefaultFileMonitor fm = new DefaultFileMonitor(new CustomFileListener());
  * fm.setRecursive(true);
- * fm.addFile(listendir);
+ * fm.addFile(listenDir);
  * fm.start();
  * </pre>
  *
@@ -77,13 +79,15 @@
 // TODO Add a Builder so we can construct and start.
 public class DefaultFileMonitor implements Runnable, FileMonitor, AutoCloseable {
 
+    private static final ThreadFactory threadFactory = new BasicThreadFactory.Builder().daemon(true).priority(Thread.MIN_PRIORITY).build();
+
     /**
      * File monitor agent.
      */
     private static final class FileMonitorAgent {
+
         private final FileObject fileObject;
         private final DefaultFileMonitor defaultFileMonitor;
-
         private boolean exists;
         private long timestamp;
         private Map<FileName, Object> children;
@@ -556,9 +560,7 @@ public void setRecursive(final boolean newRecursive) {
      */
     public synchronized void start() {
         if (monitorThread == null) {
-            monitorThread = new Thread(this);
-            monitorThread.setDaemon(true);
-            monitorThread.setPriority(Thread.MIN_PRIORITY);
+            monitorThread = threadFactory.newThread(this);
         }
         monitorThread.start();
     }