Skip to content

Commit 4733a97

Browse files
committed
do not unnecessarily create 2 File objects when returning eftar reader
fixes #539
1 parent 9113978 commit 4733a97

File tree

1 file changed

+3
-5
lines changed

1 file changed

+3
-5
lines changed

src/org/opensolaris/opengrok/configuration/Configuration.java

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -657,7 +657,7 @@ public String getHeaderIncludeFileContent() {
657657
* contains definition tags.
658658
*/
659659
public static final String EFTAR_DTAGS_FILE = "index/dtags.eftar";
660-
private transient String dtagsEftar = null;
660+
private transient File dtagsEftar = null;
661661

662662
/**
663663
* Get the eftar file, which contains definition tags.
@@ -668,12 +668,10 @@ public File getDtagsEftar() {
668668
if (dtagsEftar == null) {
669669
File tmp = new File(getDataRoot() + "/" + EFTAR_DTAGS_FILE);
670670
if (tmp.canRead()) {
671-
dtagsEftar = tmp.getName();
672-
} else {
673-
dtagsEftar = "";
671+
dtagsEftar = tmp;
674672
}
675673
}
676-
return dtagsEftar.isEmpty() ? null : new File(dtagsEftar);
674+
return dtagsEftar;
677675
}
678676

679677
public String getDatabaseDriver() {

0 commit comments

Comments
 (0)