Skip to content

Commit e76e479

Browse files
dimitrisAnyfantakisjelle-dc
authored andcommitted
Close file handle in ConfigurationWriter
1 parent 836253f commit e76e479

File tree

1 file changed

+7
-8
lines changed

1 file changed

+7
-8
lines changed

base/src/main/java/proguard/ConfigurationWriter.java

Lines changed: 7 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ public class ConfigurationWriter implements AutoCloseable
5151

5252
private final PrintWriter writer;
5353
private File configurationFile;
54-
private File baseDir;
54+
private String baseDirName;
5555

5656

5757
/**
@@ -62,7 +62,10 @@ public ConfigurationWriter(File configurationFile) throws IOException
6262
this(PrintWriterUtil.createPrintWriterOut(configurationFile));
6363

6464
this.configurationFile = configurationFile;
65-
baseDir = configurationFile.getParentFile();
65+
if (configurationFile.getParentFile() != null)
66+
{
67+
baseDirName = configurationFile.getParentFile().getAbsolutePath() + File.separator;
68+
}
6669
}
6770

6871

@@ -801,13 +804,9 @@ private String relativeFileName(File file)
801804
String fileName = file.getAbsolutePath();
802805

803806
// See if we can convert the file name into a relative file name.
804-
if (baseDir != null)
807+
if (baseDirName != null && fileName.startsWith(baseDirName))
805808
{
806-
String baseDirName = baseDir.getAbsolutePath() + File.separator;
807-
if (fileName.startsWith(baseDirName))
808-
{
809-
fileName = fileName.substring(baseDirName.length());
810-
}
809+
fileName = fileName.substring(baseDirName.length());
811810
}
812811

813812
return quotedString(fileName);

0 commit comments

Comments
 (0)