From 2a59e6ae78b1613b2e74357177433e57ec8d64f3 Mon Sep 17 00:00:00 2001
From: Marco Romano <marcor@element.io>
Date: Mon, 31 Jul 2023 15:49:57 +0200
Subject: [PATCH] Properly inline `tryOrNull()` (#1003)

Nullable params can't be inlined, default empty lambdas are therefore preferred.
---
 .../main/kotlin/io/element/android/libraries/core/data/Try.kt | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/libraries/core/src/main/kotlin/io/element/android/libraries/core/data/Try.kt b/libraries/core/src/main/kotlin/io/element/android/libraries/core/data/Try.kt
index b91d249547..fe801e71f7 100644
--- a/libraries/core/src/main/kotlin/io/element/android/libraries/core/data/Try.kt
+++ b/libraries/core/src/main/kotlin/io/element/android/libraries/core/data/Try.kt
@@ -16,11 +16,11 @@
 
 package io.element.android.libraries.core.data
 
-inline fun <A> tryOrNull(noinline onError: ((Throwable) -> Unit)? = null, operation: () -> A): A? {
+inline fun <A> tryOrNull(onError: ((Throwable) -> Unit) = { }, operation: () -> A): A? {
     return try {
         operation()
     } catch (any: Throwable) {
-        onError?.invoke(any)
+        onError.invoke(any)
         null
     }
 }