[MNG-6847] Explicit type can be replaced by the diamond operator

This closes #315
This commit is contained in:
Artem Krosheninnikov 2020-01-13 19:39:45 +03:00 committed by Michael Osipov
parent f620bb714d
commit 39492281b2
27 changed files with 41 additions and 42 deletions

View File

@ -41,7 +41,7 @@ public MultipleArtifactsNotFoundException( Artifact originatingArtifact,
List<Artifact> missingArtifacts, List<Artifact> missingArtifacts,
List<ArtifactRepository> remoteRepositories ) List<ArtifactRepository> remoteRepositories )
{ {
this( originatingArtifact, new ArrayList<Artifact>(), missingArtifacts, remoteRepositories ); this( originatingArtifact, new ArrayList<>(), missingArtifacts, remoteRepositories );
} }
/** /**

View File

@ -37,10 +37,10 @@
public class VersionRange public class VersionRange
{ {
private static final Map<String, VersionRange> CACHE_SPEC = private static final Map<String, VersionRange> CACHE_SPEC =
Collections.<String, VersionRange>synchronizedMap( new WeakHashMap<String, VersionRange>() ); Collections.synchronizedMap( new WeakHashMap<>() );
private static final Map<String, VersionRange> CACHE_VERSION = private static final Map<String, VersionRange> CACHE_VERSION =
Collections.<String, VersionRange>synchronizedMap( new WeakHashMap<String, VersionRange>() ); Collections.synchronizedMap( new WeakHashMap<>() );
private final ArtifactVersion recommendedVersion; private final ArtifactVersion recommendedVersion;

View File

@ -38,7 +38,7 @@ class DefaultProblemCollector
DefaultProblemCollector( List<Problem> problems ) DefaultProblemCollector( List<Problem> problems )
{ {
this.problems = ( problems != null ) ? problems : new ArrayList<Problem>(); this.problems = ( problems != null ) ? problems : new ArrayList<>();
} }
@Override @Override

View File

@ -116,7 +116,7 @@ public void execute( Runnable command )
else else
{ {
executor = new ThreadPoolExecutor( threads, threads, 3, TimeUnit.SECONDS, executor = new ThreadPoolExecutor( threads, threads, 3, TimeUnit.SECONDS,
new LinkedBlockingQueue<Runnable>(), new DaemonThreadCreator() ); new LinkedBlockingQueue<>(), new DaemonThreadCreator() );
} }
} }

View File

@ -45,7 +45,7 @@ public DuplicateProjectException( String message, Map<String, List<File>> collis
{ {
super( message, (File) null ); super( message, (File) null );
this.collisions = ( collisions != null ) ? collisions : new LinkedHashMap<String, List<File>>(); this.collisions = ( collisions != null ) ? collisions : new LinkedHashMap<>();
} }
/** /**

View File

@ -78,7 +78,7 @@ public MavenExecutionRequest populateFromToolchains( MavenExecutionRequest reque
{ {
if ( !groupedToolchains.containsKey( model.getType() ) ) if ( !groupedToolchains.containsKey( model.getType() ) )
{ {
groupedToolchains.put( model.getType(), new ArrayList<ToolchainModel>() ); groupedToolchains.put( model.getType(), new ArrayList<>() );
} }
groupedToolchains.get( model.getType() ).add( model ); groupedToolchains.get( model.getType() ).add( model );

View File

@ -41,7 +41,7 @@ public class DefaultMavenExecutionResult
private final List<Throwable> exceptions = new CopyOnWriteArrayList<>(); private final List<Throwable> exceptions = new CopyOnWriteArrayList<>();
private final Map<MavenProject, BuildSummary> buildSummaries = private final Map<MavenProject, BuildSummary> buildSummaries =
Collections.synchronizedMap( new IdentityHashMap<MavenProject, BuildSummary>() ); Collections.synchronizedMap( new IdentityHashMap<>() );
public MavenExecutionResult setProject( MavenProject project ) public MavenExecutionResult setProject( MavenProject project )
{ {
@ -65,7 +65,7 @@ public MavenExecutionResult setTopologicallySortedProjects( List<MavenProject> t
public List<MavenProject> getTopologicallySortedProjects() public List<MavenProject> getTopologicallySortedProjects()
{ {
return null == topologicallySortedProjects return null == topologicallySortedProjects
? Collections.<MavenProject>emptyList() ? Collections.emptyList()
: Collections.unmodifiableList( topologicallySortedProjects ); : Collections.unmodifiableList( topologicallySortedProjects );
} }

View File

@ -56,7 +56,7 @@ class FilteredProjectDependencyGraph
this.projectDependencyGraph = this.projectDependencyGraph =
Objects.requireNonNull( projectDependencyGraph, "projectDependencyGraph cannot be null" ); Objects.requireNonNull( projectDependencyGraph, "projectDependencyGraph cannot be null" );
this.whiteList = new IdentityHashMap<MavenProject, Object>(); this.whiteList = new IdentityHashMap<>();
for ( MavenProject project : whiteList ) for ( MavenProject project : whiteList )
{ {

View File

@ -179,7 +179,7 @@ public void setupMojoExecution( MavenSession session, MavenProject project, Mojo
finalizeMojoConfiguration( mojoExecution ); finalizeMojoConfiguration( mojoExecution );
calculateForkedExecutions( mojoExecution, session, project, new HashSet<MojoDescriptor>() ); calculateForkedExecutions( mojoExecution, session, project, new HashSet<>() );
} }
public List<MojoExecution> calculateMojoExecutions( MavenSession session, MavenProject project, List<Object> tasks ) public List<MojoExecution> calculateMojoExecutions( MavenSession session, MavenProject project, List<Object> tasks )
@ -331,7 +331,7 @@ public void calculateForkedExecutions( MojoExecution mojoExecution, MavenSession
PluginDescriptorParsingException, NoPluginFoundForPrefixException, InvalidPluginDescriptorException, PluginDescriptorParsingException, NoPluginFoundForPrefixException, InvalidPluginDescriptorException,
LifecyclePhaseNotFoundException, LifecycleNotFoundException, PluginVersionResolutionException LifecyclePhaseNotFoundException, LifecycleNotFoundException, PluginVersionResolutionException
{ {
calculateForkedExecutions( mojoExecution, session, session.getCurrentProject(), new HashSet<MojoDescriptor>() ); calculateForkedExecutions( mojoExecution, session, session.getCurrentProject(), new HashSet<>() );
} }
private void calculateForkedExecutions( MojoExecution mojoExecution, MavenSession session, MavenProject project, private void calculateForkedExecutions( MojoExecution mojoExecution, MavenSession session, MavenProject project,

View File

@ -62,8 +62,8 @@ public DependencyContext( MavenProject project, Collection<String> scopesToColle
this.project = project; this.project = project;
scopesToCollectForCurrentProject = scopesToCollect; scopesToCollectForCurrentProject = scopesToCollect;
scopesToResolveForCurrentProject = scopesToResolve; scopesToResolveForCurrentProject = scopesToResolve;
scopesToCollectForAggregatedProjects = Collections.synchronizedSet( new TreeSet<String>() ); scopesToCollectForAggregatedProjects = Collections.synchronizedSet( new TreeSet<>() );
scopesToResolveForAggregatedProjects = Collections.synchronizedSet( new TreeSet<String>() ); scopesToResolveForAggregatedProjects = Collections.synchronizedSet( new TreeSet<>() );
} }
public MavenProject getProject() public MavenProject getProject()

View File

@ -22,7 +22,6 @@
import java.util.HashSet; import java.util.HashSet;
import java.util.List; import java.util.List;
import org.apache.maven.artifact.Artifact;
import org.apache.maven.execution.BuildSuccess; import org.apache.maven.execution.BuildSuccess;
import org.apache.maven.execution.ExecutionEvent; import org.apache.maven.execution.ExecutionEvent;
import org.apache.maven.execution.MavenSession; import org.apache.maven.execution.MavenSession;
@ -108,7 +107,7 @@ public void buildProject( MavenSession session, MavenSession rootSession, Reacto
eventCatapult.fire( ExecutionEvent.Type.ProjectStarted, session, null ); eventCatapult.fire( ExecutionEvent.Type.ProjectStarted, session, null );
MavenExecutionPlan executionPlan = MavenExecutionPlan executionPlan =
builderCommon.resolveBuildPlan( session, currentProject, taskSegment, new HashSet<Artifact>() ); builderCommon.resolveBuildPlan( session, currentProject, taskSegment, new HashSet<>() );
List<MojoExecution> mojoExecutions = executionPlan.getMojoExecutions(); List<MojoExecution> mojoExecutions = executionPlan.getMojoExecutions();
projectExecutionListener.beforeProjectLifecycleExecution( new ProjectExecutionEvent( session, projectExecutionListener.beforeProjectLifecycleExecution( new ProjectExecutionEvent( session,

View File

@ -37,7 +37,7 @@ public class ReactorBuildStatus
{ {
private final ProjectDependencyGraph projectDependencyGraph; private final ProjectDependencyGraph projectDependencyGraph;
private final Collection<String> blackListedProjects = Collections.synchronizedSet( new HashSet<String>() ); private final Collection<String> blackListedProjects = Collections.synchronizedSet( new HashSet<>() );
private volatile boolean halted = false; private volatile boolean halted = false;

View File

@ -56,7 +56,7 @@ public class ThreadOutputMuxer
private final PrintStream defaultPrintStream = new PrintStream( defaultOutputStreamForUnknownData ); private final PrintStream defaultPrintStream = new PrintStream( defaultOutputStreamForUnknownData );
private final Set<ProjectSegment> completedBuilds = Collections.synchronizedSet( new HashSet<ProjectSegment>() ); private final Set<ProjectSegment> completedBuilds = Collections.synchronizedSet( new HashSet<>() );
private volatile ProjectSegment currentBuild; private volatile ProjectSegment currentBuild;

View File

@ -185,8 +185,8 @@ private ProjectBuildingResult build( File pomFile, ModelSource modelSource, Inte
modelProblems = result.getProblems(); modelProblems = result.getProblems();
initProject( project, Collections.<String, MavenProject>emptyMap(), true, initProject( project, Collections.emptyMap(), true,
result, new HashMap<File, Boolean>(), projectBuildingRequest ); result, new HashMap<>(), projectBuildingRequest );
} }
else if ( projectBuildingRequest.isResolveDependencies() ) else if ( projectBuildingRequest.isResolveDependencies() )
{ {
@ -374,7 +374,7 @@ public List<ProjectBuildingResult> build( List<File> pomFiles, boolean recursive
Map<String, MavenProject> projectIndex = new HashMap<>( 256 ); Map<String, MavenProject> projectIndex = new HashMap<>( 256 );
boolean noErrors = boolean noErrors =
build( results, interimResults, projectIndex, pomFiles, new LinkedHashSet<File>(), true, recursive, build( results, interimResults, projectIndex, pomFiles, new LinkedHashSet<>(), true, recursive,
config ); config );
populateReactorModelPool( modelPool, interimResults ); populateReactorModelPool( modelPool, interimResults );
@ -384,8 +384,8 @@ public List<ProjectBuildingResult> build( List<File> pomFiles, boolean recursive
try try
{ {
noErrors = noErrors =
build( results, new ArrayList<MavenProject>(), projectIndex, interimResults, request, build( results, new ArrayList<>(), projectIndex, interimResults, request,
new HashMap<File, Boolean>(), config.session ) && noErrors; new HashMap<>(), config.session ) && noErrors;
} }
finally finally
{ {
@ -465,7 +465,7 @@ private boolean build( List<ProjectBuildingResult> results, List<InterimResult>
try try
{ {
// first pass: build without building parent. // first pass: build without building parent.
initProject( project, projectIndex, false, result, new HashMap<File, Boolean>( 0 ), config.request ); initProject( project, projectIndex, false, result, new HashMap<>( 0 ), config.request );
} }
catch ( InvalidArtifactRTException iarte ) catch ( InvalidArtifactRTException iarte )
{ {

View File

@ -175,7 +175,7 @@ public class MavenProject
private DependencyFilter extensionDependencyFilter; private DependencyFilter extensionDependencyFilter;
private final Set<String> lifecyclePhases = Collections.synchronizedSet( new LinkedHashSet<String>() ); private final Set<String> lifecyclePhases = Collections.synchronizedSet( new LinkedHashSet<>() );
public MavenProject() public MavenProject()
{ {

View File

@ -220,14 +220,14 @@ public class CacheRecord
{ {
this.pomArtifact = ArtifactUtils.copyArtifact( pomArtifact ); this.pomArtifact = ArtifactUtils.copyArtifact( pomArtifact );
this.relocatedArtifact = ArtifactUtils.copyArtifactSafe( relocatedArtifact ); this.relocatedArtifact = ArtifactUtils.copyArtifactSafe( relocatedArtifact );
this.artifacts = ArtifactUtils.copyArtifacts( artifacts, new ArrayList<Artifact>() ); this.artifacts = ArtifactUtils.copyArtifacts( artifacts, new ArrayList<>() );
this.remoteRepositories = new ArrayList<>( remoteRepositories ); this.remoteRepositories = new ArrayList<>( remoteRepositories );
this.managedVersions = managedVersions; this.managedVersions = managedVersions;
if ( managedVersions != null ) if ( managedVersions != null )
{ {
this.managedVersions = this.managedVersions =
ArtifactUtils.copyArtifacts( managedVersions, new LinkedHashMap<String, Artifact>() ); ArtifactUtils.copyArtifacts( managedVersions, new LinkedHashMap<>() );
} }
File pomFile = pomArtifact.getFile(); File pomFile = pomArtifact.getFile();
@ -310,11 +310,11 @@ public ResolutionGroup get( Artifact artifact, boolean resolveManagedVersions, A
Artifact pomArtifact = ArtifactUtils.copyArtifact( cacheRecord.getArtifact() ); Artifact pomArtifact = ArtifactUtils.copyArtifact( cacheRecord.getArtifact() );
Artifact relocatedArtifact = ArtifactUtils.copyArtifactSafe( cacheRecord.getRelocatedArtifact() ); Artifact relocatedArtifact = ArtifactUtils.copyArtifactSafe( cacheRecord.getRelocatedArtifact() );
Set<Artifact> artifacts = Set<Artifact> artifacts =
ArtifactUtils.copyArtifacts( cacheRecord.getArtifacts(), new LinkedHashSet<Artifact>() ); ArtifactUtils.copyArtifacts( cacheRecord.getArtifacts(), new LinkedHashSet<>() );
Map<String, Artifact> managedVersions = cacheRecord.getManagedVersions(); Map<String, Artifact> managedVersions = cacheRecord.getManagedVersions();
if ( managedVersions != null ) if ( managedVersions != null )
{ {
managedVersions = ArtifactUtils.copyArtifacts( managedVersions, new LinkedHashMap<String, Artifact>() ); managedVersions = ArtifactUtils.copyArtifacts( managedVersions, new LinkedHashMap<>() );
} }
return new ResolutionGroup( pomArtifact, relocatedArtifact, artifacts, managedVersions, return new ResolutionGroup( pomArtifact, relocatedArtifact, artifacts, managedVersions,
cacheRecord.getRemoteRepositories() ); cacheRecord.getRemoteRepositories() );

View File

@ -129,7 +129,7 @@ Map<String, Object> retrieveContext( MavenSession session )
} }
} }
return ( context != null ) ? context : new HashMap<String, Object>(); return ( context != null ) ? context : new HashMap<>();
} }
public static final String getStorageKey( String type ) public static final String getStorageKey( String type )

View File

@ -48,7 +48,7 @@ public class DefaultToolchainsBuildingResult
public DefaultToolchainsBuildingResult( PersistedToolchains effectiveToolchains, List<Problem> problems ) public DefaultToolchainsBuildingResult( PersistedToolchains effectiveToolchains, List<Problem> problems )
{ {
this.effectiveToolchains = effectiveToolchains; this.effectiveToolchains = effectiveToolchains;
this.problems = ( problems != null ) ? problems : new ArrayList<Problem>(); this.problems = ( problems != null ) ? problems : new ArrayList<>();
} }
@Override @Override

View File

@ -45,7 +45,7 @@ public void testResolveBuildPlan()
final BuilderCommon builderCommon = getBuilderCommon(); final BuilderCommon builderCommon = getBuilderCommon();
final MavenExecutionPlan plan = final MavenExecutionPlan plan =
builderCommon.resolveBuildPlan( session1, ProjectDependencyGraphStub.A, taskSegment1, builderCommon.resolveBuildPlan( session1, ProjectDependencyGraphStub.A, taskSegment1,
new HashSet<Artifact>() ); new HashSet<>() );
assertEquals( LifecycleExecutionPlanCalculatorStub.getProjectAExceutionPlan().size(), plan.size() ); assertEquals( LifecycleExecutionPlanCalculatorStub.getProjectAExceutionPlan().size(), plan.size() );
} }

View File

@ -33,7 +33,7 @@ public class CompletionServiceStub
implements CompletionService<ProjectSegment> implements CompletionService<ProjectSegment>
{ {
List<FutureTask<ProjectSegment>> projectBuildFutureTasks = List<FutureTask<ProjectSegment>> projectBuildFutureTasks =
Collections.synchronizedList( new ArrayList<FutureTask<ProjectSegment>>() ); Collections.synchronizedList(new ArrayList<>() );
final boolean finishImmediately; final boolean finishImmediately;

View File

@ -36,7 +36,7 @@ public class MojoExecutorStub
extends MojoExecutor extends MojoExecutor
{ // This is being lazy instead of making interface { // This is being lazy instead of making interface
public List<MojoExecution> executions = Collections.synchronizedList( new ArrayList<MojoExecution>() ); public List<MojoExecution> executions = Collections.synchronizedList(new ArrayList<>() );
@Override @Override
public void execute( MavenSession session, MojoExecution mojoExecution, ProjectIndex projectIndex, public void execute( MavenSession session, MojoExecution mojoExecution, ProjectIndex projectIndex,

View File

@ -41,7 +41,7 @@ public class ConsoleMavenTransferListener
{ {
private Map<TransferResource, Long> transfers = Collections.synchronizedMap( private Map<TransferResource, Long> transfers = Collections.synchronizedMap(
new LinkedHashMap<TransferResource, Long>() ); new LinkedHashMap<>() );
private boolean printResourceNames; private boolean printResourceNames;
private int lastLength; private int lastLength;

View File

@ -437,7 +437,7 @@ else if ( !parentIds.add( parentData.getId() ) )
public ModelBuildingResult build( ModelBuildingRequest request, ModelBuildingResult result ) public ModelBuildingResult build( ModelBuildingRequest request, ModelBuildingResult result )
throws ModelBuildingException throws ModelBuildingException
{ {
return build( request, result, new LinkedHashSet<String>() ); return build( request, result, new LinkedHashSet<>() );
} }
private ModelBuildingResult build( ModelBuildingRequest request, ModelBuildingResult result, private ModelBuildingResult build( ModelBuildingRequest request, ModelBuildingResult result,

View File

@ -75,7 +75,7 @@ public void gregorianCalendarIsUsed()
String datePart = ts.replaceAll( "\\..*", "" ); String datePart = ts.replaceAll( "\\..*", "" );
/* Allow for this test running across midnight */ /* Allow for this test running across midnight */
Set<String> expected = new HashSet<String>( Arrays.asList( dateBefore, dateAfter ) ); Set<String> expected = new HashSet<>( Arrays.asList( dateBefore, dateAfter ) );
assertTrue( "Expected " + datePart + " to be in " + expected, assertTrue( "Expected " + datePart + " to be in " + expected,
expected.contains( datePart ) ); expected.contains( datePart ) );
} }

View File

@ -40,7 +40,7 @@ class DefaultSettingsBuildingResult
DefaultSettingsBuildingResult( Settings effectiveSettings, List<SettingsProblem> problems ) DefaultSettingsBuildingResult( Settings effectiveSettings, List<SettingsProblem> problems )
{ {
this.effectiveSettings = effectiveSettings; this.effectiveSettings = effectiveSettings;
this.problems = ( problems != null ) ? problems : new ArrayList<SettingsProblem>(); this.problems = ( problems != null ) ? problems : new ArrayList<>();
} }
@Override @Override

View File

@ -39,7 +39,7 @@ class DefaultSettingsProblemCollector
DefaultSettingsProblemCollector( List<SettingsProblem> problems ) DefaultSettingsProblemCollector( List<SettingsProblem> problems )
{ {
this.problems = ( problems != null ) ? problems : new ArrayList<SettingsProblem>(); this.problems = ( problems != null ) ? problems : new ArrayList<>();
} }
public List<SettingsProblem> getProblems() public List<SettingsProblem> getProblems()

View File

@ -43,9 +43,9 @@ class DefaultSettingsDecryptionResult
DefaultSettingsDecryptionResult( List<Server> servers, List<Proxy> proxies, List<SettingsProblem> problems ) DefaultSettingsDecryptionResult( List<Server> servers, List<Proxy> proxies, List<SettingsProblem> problems )
{ {
this.servers = ( servers != null ) ? servers : new ArrayList<Server>(); this.servers = ( servers != null ) ? servers : new ArrayList<>();
this.proxies = ( proxies != null ) ? proxies : new ArrayList<Proxy>(); this.proxies = ( proxies != null ) ? proxies : new ArrayList<>();
this.problems = ( problems != null ) ? problems : new ArrayList<SettingsProblem>(); this.problems = ( problems != null ) ? problems : new ArrayList<>();
} }
@Override @Override