Skip to content

Commit c014043

Browse files
committed
Cleanups
1 parent e740617 commit c014043

File tree

2 files changed

+7
-4
lines changed

2 files changed

+7
-4
lines changed

net.sourceforge.pmd.eclipse.plugin.test/src/main/java/net/sourceforge/pmd/eclipse/internal/ResourceUtil.java

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -17,19 +17,23 @@
1717
import org.eclipse.core.runtime.CoreException;
1818

1919
public final class ResourceUtil {
20+
private static final int BUFFER_SIZE = 8192;
21+
2022
private ResourceUtil() {
2123
// utility
2224
}
2325

2426
public static void copyResource(Object context, String resource, File target) throws IOException {
2527
File parent = target.getParentFile();
2628
if (parent != null && !parent.exists()) {
27-
parent.mkdirs();
29+
if (!parent.mkdirs()) {
30+
throw new IOException("Couldn't mkdirs " + parent);
31+
}
2832
}
2933
try (OutputStream out = Files.newOutputStream(target.toPath());
3034
InputStream in = context.getClass().getResourceAsStream(resource)) {
3135
int count;
32-
byte[] buffer = new byte[8192];
36+
byte[] buffer = new byte[BUFFER_SIZE];
3337
count = in.read(buffer);
3438
while (count > -1) {
3539
out.write(buffer, 0, count);
@@ -41,7 +45,7 @@ public static void copyResource(Object context, String resource, File target) th
4145
public static String getResourceAsString(IProject project, String resourceName) throws IOException, CoreException {
4246
IFile file = project.getFile(resourceName);
4347
String charset = file.getCharset();
44-
char[] buffer = new char[1024];
48+
char[] buffer = new char[BUFFER_SIZE];
4549
StringBuilder result = new StringBuilder();
4650
try (Reader in = new InputStreamReader(file.getContents(), charset)) {
4751
int count = in.read(buffer);

net.sourceforge.pmd.eclipse.plugin.test/src/main/java/net/sourceforge/pmd/eclipse/runtime/cmd/MultipleRulesetsTest.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,6 @@ public void setUp() throws Exception {
4545
this.testProject != null && this.testProject.exists() && this.testProject.isAccessible());
4646

4747
// 2. Setup test folder
48-
this.testProject.getFolder("src/main/java").getFullPath().toFile().mkdirs();
4948
IFolder folder = this.testProject.getFolder("/src/main");
5049
folder.create(true, true, null);
5150
folder = folder.getFolder("java");

0 commit comments

Comments
 (0)