From 3d34728c4bc0936bec0411af98e08f53317d0061 Mon Sep 17 00:00:00 2001 From: Uwe Schindler Date: Sun, 21 Feb 2016 16:39:13 +0100 Subject: [PATCH] Prevent warning on empty try-with-resources --- .../java/org/apache/lucene/store/BaseDirectoryTestCase.java | 3 +++ .../java/org/apache/lucene/store/MockDirectoryWrapper.java | 4 +--- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/lucene/test-framework/src/java/org/apache/lucene/store/BaseDirectoryTestCase.java b/lucene/test-framework/src/java/org/apache/lucene/store/BaseDirectoryTestCase.java index 5f66074718e..44c03ea4b7e 100644 --- a/lucene/test-framework/src/java/org/apache/lucene/store/BaseDirectoryTestCase.java +++ b/lucene/test-framework/src/java/org/apache/lucene/store/BaseDirectoryTestCase.java @@ -1213,6 +1213,7 @@ public abstract class BaseDirectoryTestCase extends LuceneTestCase { continue; } try (IndexOutput out = dir.createOutput(candidate, IOContext.DEFAULT)) { + out.getFilePointer(); // just fake access to prevent compiler warning } fsDir.deleteFile(candidate); if (fsDir.checkPendingDeletions()) { @@ -1246,12 +1247,14 @@ public abstract class BaseDirectoryTestCase extends LuceneTestCase { if (random().nextBoolean()) { try (IndexOutput out = fsDir.createOutput(fileName + "z", IOContext.DEFAULT)) { + out.getFilePointer(); // just fake access to prevent compiler warning } // Make sure we can rename onto the deleted file: fsDir.renameFile(fileName + "z", fileName); } else { // write the file again try (IndexOutput out = dir.createOutput(fileName, IOContext.DEFAULT)) { + out.getFilePointer(); // just fake access to prevent compiler warning } } assertEquals(0, fsDir.fileLength(fileName)); diff --git a/lucene/test-framework/src/java/org/apache/lucene/store/MockDirectoryWrapper.java b/lucene/test-framework/src/java/org/apache/lucene/store/MockDirectoryWrapper.java index ef4397f40af..962062e0312 100644 --- a/lucene/test-framework/src/java/org/apache/lucene/store/MockDirectoryWrapper.java +++ b/lucene/test-framework/src/java/org/apache/lucene/store/MockDirectoryWrapper.java @@ -19,7 +19,6 @@ package org.apache.lucene.store; import java.io.Closeable; import java.io.FileNotFoundException; import java.io.IOException; -import java.nio.file.FileSystem; import java.nio.file.NoSuchFileException; import java.util.ArrayList; import java.util.Arrays; @@ -46,8 +45,6 @@ import org.apache.lucene.index.IndexWriter; import org.apache.lucene.index.IndexWriterConfig; import org.apache.lucene.index.NoDeletionPolicy; import org.apache.lucene.index.SegmentInfos; -import org.apache.lucene.mockfile.FilterFileSystem; -import org.apache.lucene.mockfile.VirusCheckingFS; import org.apache.lucene.util.IOUtils; import org.apache.lucene.util.LuceneTestCase; import org.apache.lucene.util.TestUtil; @@ -423,6 +420,7 @@ public class MockDirectoryWrapper extends BaseDirectoryWrapper { deleteFile(name); try (IndexOutput out = in.createOutput(name, LuceneTestCase.newIOContext(randomState))) { + out.getFilePointer(); // just fake access to prevent compiler warning } catch (IOException ioe) { // ignore }