Moved logic for handling extra calls to dedicated class "OneOffDelegate"

*This commit is related to issue #529 [1]*

[1] #529

data/src/main/java/org/cryptomator/data/db/sqlmapping/MappingSupportSQLiteDatabase.kt

@@ -164,18 +164,11 @@ internal class MappingSupportSQLiteDatabase(
 		private val delegateQuery: SupportSQLiteQuery
 	) : SupportSQLiteQuery by delegateQuery {
 
-		private val lock = Any()
-		private var called = false
-
 		private val _sql = map(delegateQuery.sql)
+		private val sqlDelegate = OneOffDelegate { Timber.tag("MappingSupportSQLiteQuery").e("SQL queried twice") }
+
 		override val sql: String
-			get() = synchronized(lock) {
-				if (called) {
-					Timber.tag("MappingSupportSQLiteQuery").e("SQL queried twice")
-				}
-				called = true
-				return _sql
-			}
+			get() = sqlDelegate.call { _sql }
 	}
 }
 

data/src/main/java/org/cryptomator/data/db/sqlmapping/OneOffDelegate.kt

@@ -0,0 +1,15 @@
+package org.cryptomator.data.db.sqlmapping
+
+internal class OneOffDelegate(private val beforeExtraCalls: () -> Unit) {
+
+	private val lock = Any()
+	private var called = false
+
+	fun <R> call(delegateCallable: () -> R): R = synchronized(lock) {
+		if (called) {
+			beforeExtraCalls()
+		}
+		called = true
+		return delegateCallable()
+	}
+}