Skip to content

Commit 0bb8796

Browse files
committed
refactor: use android log in cleanup
1 parent dee14a9 commit 0bb8796

1 file changed

Lines changed: 10 additions & 6 deletions

File tree

app/src/main/java/to/bitkit/utils/Logger.kt

Lines changed: 10 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -239,14 +239,18 @@ class LogSaverImpl(
239239
}
240240
}
241241

242-
private fun log(message: String, level: LogLevel = LogLevel.INFO) {
242+
private fun log(
243+
message: String,
244+
level: LogLevel = LogLevel.INFO,
245+
androidLog: (String, String) -> Unit = { tag, msg -> Log.i(tag, msg) },
246+
) {
243247
val formatted = formatLog(level, message, TAG, getCallerPath(), getCallerLine())
244-
Log.i(APP, formatted)
248+
androidLog(APP, formatted)
245249
save(formatted)
246250
}
247251

248252
private fun cleanupOldLogFiles(maxTotalSizeMB: Int = 20) {
249-
log("Deleting old log files…", LogLevel.VERBOSE)
253+
log("Deleting old log files…", LogLevel.VERBOSE, Log::v)
250254
val logDir = runCatching { Env.logDir }.getOrNull() ?: return
251255

252256
val logFiles = logDir
@@ -263,15 +267,15 @@ class LogSaverImpl(
263267
if (totalSize <= maxSizeBytes) return
264268

265269
runCatching {
266-
Log.d(APP, "Deleting old log file: '${file.name}'")
270+
log("Deleting old log file: '${file.name}'", LogLevel.DEBUG, Log::d)
267271
if (file.delete()) {
268272
totalSize -= file.length()
269273
}
270274
}.onFailure {
271-
Log.w(APP, "Failed to delete old log file: '${file.name}'", it)
275+
log("Failed to delete old log file: '${file.name}'", LogLevel.WARN, Log::w)
272276
}
273277
}
274-
Log.v(APP, "Deleted all old log files.")
278+
log("Deleted all old log files.", LogLevel.VERBOSE, Log::v)
275279
}
276280

277281
companion object {

0 commit comments

Comments
 (0)