Fixed style issues
git-svn-id: https://svn.apache.org/repos/asf/maven/shared/trunk@1632627 13f79535-47bb-0310-9956-ffa450edef68
diff --git a/src/main/java/org/apache/maven/shared/filtering/DefaultMavenResourcesFiltering.java b/src/main/java/org/apache/maven/shared/filtering/DefaultMavenResourcesFiltering.java
index 2821619..0b2d6ce 100644
--- a/src/main/java/org/apache/maven/shared/filtering/DefaultMavenResourcesFiltering.java
+++ b/src/main/java/org/apache/maven/shared/filtering/DefaultMavenResourcesFiltering.java
@@ -182,8 +182,8 @@
if ( getLogger().isDebugEnabled() )
{
String ls = System.getProperty( "line.separator" );
- StringBuffer debugMessage =
- new StringBuffer( "resource with targetPath " ).append( resource.getTargetPath() ).append( ls );
+ StringBuilder debugMessage =
+ new StringBuilder( "resource with targetPath " ).append( resource.getTargetPath() ).append( ls );
debugMessage.append( "directory " ).append( resource.getDirectory() ).append( ls );
debugMessage.append( "excludes " ).append( resource.getExcludes() == null ? " empty "
: resource.getExcludes().toString() ).append( ls );
diff --git a/src/main/java/org/apache/maven/shared/filtering/FilteringUtils.java b/src/main/java/org/apache/maven/shared/filtering/FilteringUtils.java
index 6a2928a..8a5d9cf 100644
--- a/src/main/java/org/apache/maven/shared/filtering/FilteringUtils.java
+++ b/src/main/java/org/apache/maven/shared/filtering/FilteringUtils.java
@@ -48,7 +48,7 @@
if ( !StringUtils.isEmpty( val ) && PATTERN.matcher( val ).matches() )
{
// Adapted from StringUtils.replace in plexus-utils to accommodate pre-escaped backslashes.
- StringBuffer buf = new StringBuffer( val.length() );
+ StringBuilder buf = new StringBuilder( val.length() );
int start = 0, end = 0;
while ( ( end = val.indexOf( '\\', start ) ) != -1 )
{
diff --git a/src/main/java/org/apache/maven/shared/filtering/InterpolatorFilterReaderLineEnding.java b/src/main/java/org/apache/maven/shared/filtering/InterpolatorFilterReaderLineEnding.java
index dcc7326..bfa717f 100644
--- a/src/main/java/org/apache/maven/shared/filtering/InterpolatorFilterReaderLineEnding.java
+++ b/src/main/java/org/apache/maven/shared/filtering/InterpolatorFilterReaderLineEnding.java
@@ -232,7 +232,7 @@
boolean inEscape = ( useEscape && ch == escapeString.charAt( 0 ) );
- StringBuffer key = new StringBuffer();
+ StringBuilder key = new StringBuilder();
// have we found an escape string?
if ( inEscape )
diff --git a/src/main/java/org/apache/maven/shared/filtering/MultiDelimiterInterpolatorFilterReaderLineEnding.java b/src/main/java/org/apache/maven/shared/filtering/MultiDelimiterInterpolatorFilterReaderLineEnding.java
index 9c0eca8..ee708ca 100644
--- a/src/main/java/org/apache/maven/shared/filtering/MultiDelimiterInterpolatorFilterReaderLineEnding.java
+++ b/src/main/java/org/apache/maven/shared/filtering/MultiDelimiterInterpolatorFilterReaderLineEnding.java
@@ -251,7 +251,7 @@
boolean inEscape = ( useEscape && ch == escapeString.charAt( 0 ) );
- StringBuffer key = new StringBuffer();
+ StringBuilder key = new StringBuilder();
// have we found an escape string?
if ( inEscape )
diff --git a/src/main/java/org/apache/maven/shared/filtering/PropertyUtils.java b/src/main/java/org/apache/maven/shared/filtering/PropertyUtils.java
index 81741e7..291663b 100644
--- a/src/main/java/org/apache/maven/shared/filtering/PropertyUtils.java
+++ b/src/main/java/org/apache/maven/shared/filtering/PropertyUtils.java
@@ -87,9 +87,9 @@
// as can be verified by replacing the implementation of #loadPropertyFile(File, boolean, boolean)
// with the commented variant I have provided that reuses this method.
- for ( Iterator<Object> iter = fileProps.keySet().iterator(); iter.hasNext(); )
+ for ( Object o : fileProps.keySet() )
{
- final String k = (String) iter.next();
+ final String k = (String) o;
final String propValue = getPropertyValue( k, combinedProps );
fileProps.setProperty( k, propValue );
}
diff --git a/src/test/java/org/apache/maven/shared/filtering/DefaultMavenResourcesFilteringTest.java b/src/test/java/org/apache/maven/shared/filtering/DefaultMavenResourcesFilteringTest.java
index 9706ddc..27263eb 100644
--- a/src/test/java/org/apache/maven/shared/filtering/DefaultMavenResourcesFilteringTest.java
+++ b/src/test/java/org/apache/maven/shared/filtering/DefaultMavenResourcesFilteringTest.java
@@ -48,6 +48,7 @@
File outputDirectory = new File( getBasedir(), "target/DefaultMavenResourcesFilteringTest" );
+ @SuppressWarnings( "ResultOfMethodCallIgnored" )
protected void setUp()
throws Exception
{
@@ -221,6 +222,7 @@
assertFiltering( baseDir, initialImageFile, true, true );
}
+ @SuppressWarnings( "ConstantConditions" )
private void assertFiltering( File baseDir, File initialImageFile, boolean escapeTest, boolean additionnalProperties )
throws Exception
{
@@ -484,6 +486,7 @@
mavenResourcesFiltering.filterResources( mavenResourcesExecution );
File[] files = outputDirectory.listFiles();
+ assertNotNull( files);
assertEquals( 2, files.length );
File includeFile = new File( outputDirectory, "includefile.txt" );
assertTrue( includeFile.exists() );
@@ -650,11 +653,11 @@
mavenResourcesFiltering.filterResources( mavenResourcesExecution );
File[] childs = outputDirectory.listFiles();
+ assertNotNull( childs );
assertEquals( 3, childs.length );
- for ( int i = 0, size = childs.length; i < size; i++ )
+ for ( File file : childs )
{
- File file = childs[i];
if ( file.getName().endsWith( "dir1" ) || file.getName().endsWith( "empty-directory" )
|| file.getName().endsWith( "empty-directory-child" ) )
{
diff --git a/src/test/java/org/apache/maven/shared/filtering/EscapeStringTest.java b/src/test/java/org/apache/maven/shared/filtering/EscapeStringTest.java
index 033ad87..d00cce1 100644
--- a/src/test/java/org/apache/maven/shared/filtering/EscapeStringTest.java
+++ b/src/test/java/org/apache/maven/shared/filtering/EscapeStringTest.java
@@ -95,8 +95,10 @@
String content = IOUtil.toString( new FileInputStream( new File( outputDirectory, "content.xml" ) ) );
System.out.println( "content " + content );
- assertTrue( content.indexOf( "<broken-tag>Content with replacement: I am the replacement !</broken-tag>" ) >= 0 );
- assertTrue( content.indexOf( "<broken-tag>Content with escaped replacement: Do not ${replaceThis} !</broken-tag>" ) >= 0 );
+ assertTrue(
+ content.contains( "<broken-tag>Content with replacement: I am the replacement !</broken-tag>" ) );
+ assertTrue( content.contains(
+ "<broken-tag>Content with escaped replacement: Do not ${replaceThis} !</broken-tag>" ) );
}
finally
{
diff --git a/src/test/java/org/apache/maven/shared/filtering/IncrementalResourceFilteringTest.java b/src/test/java/org/apache/maven/shared/filtering/IncrementalResourceFilteringTest.java
index 2cd189e..1e89f6b 100644
--- a/src/test/java/org/apache/maven/shared/filtering/IncrementalResourceFilteringTest.java
+++ b/src/test/java/org/apache/maven/shared/filtering/IncrementalResourceFilteringTest.java
@@ -45,7 +45,7 @@
File unitDirectory = new File( getBasedir(), "src/test/units-files/incremental" );
- protected void setUp()
+ @SuppressWarnings( "ResultOfMethodCallIgnored" ) protected void setUp()
throws Exception
{
super.setUp();
@@ -78,10 +78,6 @@
assertTrue( ctx.getRefreshFiles().contains( new File( outputDirectory, "file01.txt" ) ) );
- // one file is expected to be deleted
- Set<String> deletedFiles = new HashSet<String>();
- deletedFiles.add( "file01.txt" );
-
ctx = new TestIncrementalBuildContext( unitDirectory, new HashSet(), changedFiles, new HashMap() );
ThreadBuildContext.setThreadBuildContext( ctx );
@@ -175,7 +171,7 @@
}
private void filter( String time )
- throws Exception, MavenFilteringException
+ throws Exception
{
File baseDir = new File( getBasedir() );
StubMavenProject mavenProject = new StubMavenProject( baseDir );
@@ -202,7 +198,7 @@
filtersFile.add( new File( unitDirectory, "filters.txt" ).getPath() );
mavenResourcesFiltering.filterResources( resources, outputDirectory, mavenProject, "UTF-8", filtersFile,
- new ArrayList(), new StubMavenSession() );
+ new ArrayList<String>(), new StubMavenSession() );
}
}
