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.
      */

Reply via email to