Skip to content

Commit 9c0a2a8

Browse files
Vladimir Kotalahornace
authored andcommitted
make renamedFiles Set
1 parent fe571d1 commit 9c0a2a8

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

opengrok-indexer/src/main/java/org/opengrok/indexer/history/MercurialHistoryParser.java

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -33,8 +33,10 @@
3333
import java.text.ParseException;
3434
import java.util.ArrayList;
3535
import java.util.Date;
36+
import java.util.HashSet;
3637
import java.util.Iterator;
3738
import java.util.List;
39+
import java.util.Set;
3840
import java.util.logging.Level;
3941
import java.util.logging.Logger;
4042
import org.opengrok.indexer.configuration.RuntimeEnvironment;
@@ -56,7 +58,7 @@ class MercurialHistoryParser implements Executor.StreamHandler {
5658
private final MercurialRepository repository;
5759
private final String mydir;
5860
private boolean isDir;
59-
private final List<String> renamedFiles = new ArrayList<>();
61+
private final Set<String> renamedFiles = new HashSet<>();
6062

6163
MercurialHistoryParser(MercurialRepository repository) {
6264
this.repository = repository;
@@ -184,8 +186,7 @@ public void processStream(InputStream input) throws IOException {
184186
*/
185187
String[] move = part.split(" \\(");
186188
File f = new File(mydir + move[0]);
187-
if (!move[0].isEmpty() && f.exists() &&
188-
!renamedFiles.contains(repository.getDirectoryNameRelative() + File.separator + move[0])) {
189+
if (!move[0].isEmpty() && f.exists()) {
189190
renamedFiles.add(repository.getDirectoryNameRelative() + File.separator + move[0]);
190191
}
191192
}

0 commit comments

Comments
 (0)