Skip to content

Commit e7bf9b5

Browse files
committed
cleanup the way of getting the s. file
1 parent 5c10bf7 commit e7bf9b5

File tree

2 files changed

+6
-2
lines changed

2 files changed

+6
-2
lines changed

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

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,7 @@
2727
import java.io.FileReader;
2828
import java.io.IOException;
2929
import java.io.Reader;
30+
import java.nio.file.Paths;
3031
import java.text.ParseException;
3132
import java.util.ArrayList;
3233
import java.util.Date;
@@ -254,7 +255,7 @@ private static File getSCCSFile(File file) {
254255
}
255256

256257
static File getSCCSFile(String parent, String name) {
257-
File f = new File(parent + "/SCCS/s." + name);
258+
File f = Paths.get(parent, "SCCS", "s." + name).toFile();
258259
if (!f.exists()) {
259260
return null;
260261
}

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

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -148,7 +148,10 @@ public boolean fileHasAnnotation(File file) {
148148
public boolean fileHasHistory(File file) {
149149
String parentFile = file.getParent();
150150
String name = file.getName();
151-
File f = Paths.get(parentFile, "SCCS", "s." + name).toFile();
151+
File f = SCCSHistoryParser.getSCCSFile(parentFile, name);
152+
if (f == null) {
153+
return false;
154+
}
152155
return f.exists();
153156
}
154157

0 commit comments

Comments
 (0)