fix: Use ifThenElse_void for Result as well

Author: MarcellPerger1

src/main/java/net/marcellperger/mathexpr/util/rs/Result.java

@@ -110,7 +110,9 @@ default <E2> Result<T, E2> mapErr(Function<? super E, ? extends E2> op) {
     }
 
     // not a Rust function. Similar to mapOrElse but reversed arguments
-    default void ifThenElse(Consumer<? super T> okFn, Consumer<? super E> errFn) {
+    default void ifThenElse_void(Consumer<? super T> okFn, Consumer<? super E> errFn) {
+        // This _void variant exists (instead of an overload) to avoid Java complaining
+        // about ambiguous arguments when passing method references
         mapOrElse(Util.consumerToFunction(errFn), Util.consumerToFunction(okFn));
     }
     default <U> U ifThenElse(Function<? super T, ? extends U> okFn, Function<? super E, ? extends U> errFn) {

src/test/java/net/marcellperger/mathexpr/util/rs/ResultTest.java

@@ -124,18 +124,18 @@ void mapErr() {
     }
 
     @Test
-    void ifThenElse_consumer() {
+    void ifThenElse_void() {
         MockedConsumer<Integer> intConsumer = new MockedConsumer<>();
         MockedConsumer<String> strConsumer = new MockedConsumer<>();
         {
-            getOk().ifThenElse(intConsumer, strConsumer);
+            getOk().ifThenElse_void(intConsumer, strConsumer);
             intConsumer.assertCalledOnceWith(314);
             strConsumer.assertNotCalled();
         }
         intConsumer.reset();
         strConsumer.reset();
         {
-            getErr().ifThenElse(intConsumer, strConsumer);
+            getErr().ifThenElse_void(intConsumer, strConsumer);
             intConsumer.assertNotCalled();
             strConsumer.assertCalledOnceWith("TESTING_ERROR");
         }