/*
 * JBoss, Home of Professional Open Source.
 * Copyright 2010, Red Hat, Inc., and individual contributors
 * as indicated by the @author tags. See the copyright.txt file in the
 * distribution for a full listing of individual contributors.
 *
 * This is free software; you can redistribute it and/or modify it
 * under the terms of the GNU Lesser General Public License as
 * published by the Free Software Foundation; either version 2.1 of
 * the License, or (at your option) any later version.
 *
 * This software is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
 * Lesser General Public License for more details.
 *
 * You should have received a copy of the GNU Lesser General Public
 * License along with this software; if not, write to the Free
 * Software Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA
 * 02110-1301 USA, or see the FSF site: http://www.fsf.org.
 */

package org.jboss.as.deployment.module;

import org.jboss.msc.service.Service;
import org.jboss.msc.service.ServiceName;
import org.jboss.msc.service.StartContext;
import org.jboss.msc.service.StartException;
import org.jboss.msc.service.StopContext;
import org.jboss.vfs.TempFileProvider;
import org.jboss.vfs.VFSUtils;

import java.io.IOException;
import java.util.concurrent.Executors;

/**
 * Service responsible for managing the life-cycle of a TempFileProvider.
 *
 * @author John E. Bailey
 */
public class TempFileProviderService implements Service<TempFileProvider> {
    public static final ServiceName SERVICE_NAME = ServiceName.JBOSS.append("TempFileProvider");

    private static TempFileProvider PROVIDER;
    private static final Object LOCK = new Object();

    /**
     * {@inheritDoc}
     */
    public void start(StartContext context) throws StartException {
    }

    /**
     * {@inheritDoc}
     */
    public void stop(StopContext context) {
        VFSUtils.safeClose(PROVIDER);
    }

    /**
     * {@inheritDoc}
     */
    public TempFileProvider getValue() throws IllegalStateException {
        return provider();
    }

    public static TempFileProvider provider() {
        synchronized(LOCK) {
            if (PROVIDER == null) {
                try {
                    PROVIDER = TempFileProvider.create("deployment", Executors.newScheduledThreadPool(2));
                } catch (IOException e) {
                    throw new RuntimeException("Failed to create temp file provider");
                }
            }
        }
        return PROVIDER;
    }
}

@@ -1703,32 +1703,38 @@
  private static 
+final 
 TempFileProvider
@@ -1752,56 +1752,284 @@
     
-private static final Object LOCK = new Object();
+static%0A    %7B%0A       try%0A       %7B%0A          PROVIDER = TempFileProvider.create(%22deployment%22, Executors.newScheduledThreadPool(2));%0A       %7D%0A       catch (final IOException ioe)%0A       %7B%0A          throw new RuntimeException(%22Failed to create temp file provider%22);%0A       %7D%0A    %7D
 %0A%0A  
@@ -2401,24 +2401,24 @@
 r();%0A    %7D%0A%0A
+
     public s
@@ -2457,373 +2457,8 @@
 ) %7B%0A
-        synchronized(LOCK) %7B%0A            if (PROVIDER == null) %7B%0A                try %7B%0A                    PROVIDER = TempFileProvider.create(%22deployment%22, Executors.newScheduledThreadPool(2));%0A                %7D catch (IOException e) %7B%0A                    throw new RuntimeException(%22Failed to create temp file provider%22);%0A                %7D%0A            %7D%0A        %7D%0A
     

Remove unnecessary lock acquisition