Merge pull request #39 from apache/close

use try with resources instead of deprecated method

Author: elharo

src/main/java/org/apache/maven/plugins/checkstyle/exec/DefaultCheckstyleExecutor.java

@@ -38,7 +38,6 @@
 import java.util.Properties;
 import java.util.Set;
 
-import org.apache.commons.io.IOUtils;
 import org.apache.maven.artifact.Artifact;
 import org.apache.maven.artifact.DependencyResolutionRequiredException;
 import org.apache.maven.model.Resource;
@@ -496,7 +495,6 @@ private Properties getOverridingProperties( CheckstyleExecutorRequest request )
         throws CheckstyleExecutorException
     {
         Properties p = new Properties();
-        InputStream in = null;
         try
         {
             if ( request.getPropertiesLocation() != null )
@@ -511,10 +509,10 @@ private Properties getOverridingProperties( CheckstyleExecutorRequest request )
 
                 if ( propertiesFile != null )
                 {
-                    in = new FileInputStream( propertiesFile );
-                    p.load( in );
-                    in.close();
-                    in = null;
+                    try ( InputStream in = new FileInputStream( propertiesFile ) )
+                    {
+                        p.load( in );
+                    }
                 }
             }
 
@@ -570,10 +568,6 @@ private Properties getOverridingProperties( CheckstyleExecutorRequest request )
         {
             throw new CheckstyleExecutorException( "Failed to get overriding properties", e );
         }
-        finally
-        {
-            IOUtils.closeQuietly( in );
-        }
         if ( request.getSuppressionsFileExpression() != null )
         {
             String suppressionsFilePath = getSuppressionsFilePath( request );