Skip to content

Commit

Permalink
Some more cleanup
Browse files Browse the repository at this point in the history
  • Loading branch information
Alexandre Fillatre committed Apr 3, 2022
1 parent 66eb70f commit 29ed30a
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ public void write() throws IOException {
log.info("Starting debloating file");
logChanges();
setDependencies(analysis.getDebloatedDependencies().stream()
.map(this::toMavenDependency)
.map(this::toProviderDependency)
.collect(Collectors.toList()));

if (log.isDebugEnabled()) {
Expand All @@ -34,7 +34,7 @@ public void write() throws IOException {
writeFile();
}

protected abstract T toMavenDependency(DebloatedDependency debloatedDependency);
protected abstract T toProviderDependency(DebloatedDependency debloatedDependency);

protected abstract void setDependencies(List<T> dependencies);

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@

import java.io.File;
import java.io.IOException;
import java.io.Serializable;
import java.nio.file.Path;
import java.util.Set;
import se.kth.depclean.core.AbstractDebloater;
Expand Down Expand Up @@ -70,7 +69,7 @@ public interface DependencyManagerWrapper {
* @param analysis the depclean analysis
* @return the debloater
*/
AbstractDebloater<? extends Serializable> getDebloater(ProjectDependencyAnalysis analysis);
AbstractDebloater<?> getDebloater(ProjectDependencyAnalysis analysis);

/**
* The build directory path.
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@ protected void logDependencies() {
}

@Override
protected Dependency toMavenDependency(DebloatedDependency debloatedDependency) {
protected Dependency toProviderDependency(DebloatedDependency debloatedDependency) {
final Dependency dependency = createDependency(debloatedDependency);
debloatedDependency.getExclusions().forEach(depToExclude -> exclude(dependency, depToExclude));
return dependency;
Expand Down

0 comments on commit 29ed30a

Please sign in to comment.