This is an automated email from the ASF dual-hosted git repository. ggregory pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/commons-configuration.git
commit e53dd2b89f5a6aee4cc38a55fa9b9064754d84c0 Author: Gary Gregory <garydgreg...@gmail.com> AuthorDate: Mon Apr 7 09:22:40 2025 -0400 Sort members --- .../org/apache/commons/configuration2/AbstractConfiguration.java | 8 ++++---- .../org/apache/commons/configuration2/io/TestFileHandler.java | 8 ++++---- 2 files changed, 8 insertions(+), 8 deletions(-) diff --git a/src/main/java/org/apache/commons/configuration2/AbstractConfiguration.java b/src/main/java/org/apache/commons/configuration2/AbstractConfiguration.java index ffc8e40d..eb198664 100644 --- a/src/main/java/org/apache/commons/configuration2/AbstractConfiguration.java +++ b/src/main/java/org/apache/commons/configuration2/AbstractConfiguration.java @@ -1521,19 +1521,19 @@ public abstract class AbstractConfiguration extends BaseEventSource implements C return new SubsetConfiguration(this, prefix, DELIMITER); } - void syncRead(final Runnable runnable, final boolean optimize) { + <T, E extends Throwable> T syncRead(final FailableSupplier<T, E> supplier, final boolean optimize) throws E { beginRead(optimize); try { - runnable.run(); + return supplier.get(); } finally { endRead(); } } - <T, E extends Throwable> T syncRead(final FailableSupplier<T, E> supplier, final boolean optimize) throws E { + void syncRead(final Runnable runnable, final boolean optimize) { beginRead(optimize); try { - return supplier.get(); + runnable.run(); } finally { endRead(); } diff --git a/src/test/java/org/apache/commons/configuration2/io/TestFileHandler.java b/src/test/java/org/apache/commons/configuration2/io/TestFileHandler.java index 10ec8bb1..5fca03ea 100644 --- a/src/test/java/org/apache/commons/configuration2/io/TestFileHandler.java +++ b/src/test/java/org/apache/commons/configuration2/io/TestFileHandler.java @@ -302,6 +302,10 @@ public class TestFileHandler { }); } + private Path createTestPath() { + return createTestFile().toPath(); + } + private URI createTestURI() { return createTestFile().toURI(); } @@ -682,10 +686,6 @@ public class TestFileHandler { assertEquals(CONTENT, content.getContent()); } - private Path createTestPath() { - return createTestFile().toPath(); - } - /** * Tests whether data from a URL can be loaded. */