svn: Refactored CheckinClient - used List<File> instead of Collection<File> for method arguments

This commit is contained in:
Konstantin Kolosovsky
2015-03-24 13:06:22 +03:00
parent a1ebf9c12e
commit 5c0f5bb506
3 changed files with 6 additions and 6 deletions
@@ -20,7 +20,7 @@ import org.jetbrains.annotations.NotNull;
import org.jetbrains.idea.svn.api.SvnClient;
import java.io.File;
import java.util.Collection;
import java.util.List;
/**
* @author Konstantin Kolosovsky.
@@ -28,5 +28,5 @@ import java.util.Collection;
public interface CheckinClient extends SvnClient {
@NotNull
CommitInfo[] commit(@NotNull Collection<File> paths, @NotNull String comment) throws VcsException;
CommitInfo[] commit(@NotNull List<File> paths, @NotNull String comment) throws VcsException;
}
@@ -58,7 +58,7 @@ public class CmdCheckinClient extends BaseSvnClient implements CheckinClient {
@NotNull
@Override
public CommitInfo[] commit(@NotNull Collection<File> paths, @NotNull String message) throws VcsException {
public CommitInfo[] commit(@NotNull List<File> paths, @NotNull String message) throws VcsException {
// if directory renames were used, IDEA reports all files under them as moved, but for svn we can not pass some of them
// to commit command - since not all paths are registered as changes -> so we need to filter these cases, but only if
// there at least some child-parent relationships in passed paths
@@ -102,7 +102,7 @@ public class CmdCheckinClient extends BaseSvnClient implements CheckinClient {
return revision;
}
private Collection<File> filterCommittables(@NotNull Collection<File> committables) throws SvnBindException {
private List<File> filterCommittables(@NotNull List<File> committables) throws SvnBindException {
final Set<String> childrenOfSomebody = ContainerUtil.newHashSet();
new AbstractFilterChildren<File>() {
@Override
@@ -32,7 +32,7 @@ import org.tmatesoft.svn.core.wc.SVNCommitClient;
import org.tmatesoft.svn.core.wc.SVNCommitPacket;
import java.io.File;
import java.util.Collection;
import java.util.List;
/**
* @author Konstantin Kolosovsky.
@@ -43,7 +43,7 @@ public class SvnKitCheckinClient extends BaseSvnClient implements CheckinClient
@NotNull
@Override
public CommitInfo[] commit(@NotNull Collection<File> paths, @NotNull String comment) throws VcsException {
public CommitInfo[] commit(@NotNull List<File> paths, @NotNull String comment) throws VcsException {
File[] pathsToCommit = ArrayUtil.toObjectArray(paths, File.class);
boolean keepLocks = myVcs.getSvnConfiguration().isKeepLocks();
SVNCommitPacket[] commitPackets = null;