Skip to content

Commit 15f250e

Browse files
author
Gary Gregory
committed
Use final
1 parent 7949b84 commit 15f250e

1 file changed

Lines changed: 7 additions & 8 deletions

File tree

src/main/java/org/apache/commons/io/output/LockableFileWriter.java

Lines changed: 7 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -123,14 +123,13 @@ public LockableFileWriter(final File file, final Charset charset) throws IOExcep
123123
* @throws IOException in case of an I/O error
124124
* @since 2.3
125125
*/
126-
public LockableFileWriter(File file, final Charset charset, final boolean append,
127-
String lockDir) throws IOException {
126+
public LockableFileWriter(final File file, final Charset charset, final boolean append, String lockDir) throws IOException {
128127
// init file to create/append
129-
file = file.getAbsoluteFile();
130-
if (file.getParentFile() != null) {
131-
FileUtils.forceMkdir(file.getParentFile());
128+
final File absFile = file.getAbsoluteFile();
129+
if (absFile.getParentFile() != null) {
130+
FileUtils.forceMkdir(absFile.getParentFile());
132131
}
133-
if (file.isDirectory()) {
132+
if (absFile.isDirectory()) {
134133
throw new IOException("File specified is a directory");
135134
}
136135

@@ -141,13 +140,13 @@ public LockableFileWriter(File file, final Charset charset, final boolean append
141140
final File lockDirFile = new File(lockDir);
142141
FileUtils.forceMkdir(lockDirFile);
143142
testLockDir(lockDirFile);
144-
lockFile = new File(lockDirFile, file.getName() + LCK);
143+
lockFile = new File(lockDirFile, absFile.getName() + LCK);
145144

146145
// check if locked
147146
createLock();
148147

149148
// init wrapped writer
150-
out = initWriter(file, charset, append);
149+
out = initWriter(absFile, charset, append);
151150
}
152151

153152
/**

0 commit comments

Comments
 (0)