Ignore LogConditional issues (#3615)
Logs are stripped in release builds, this is an unnecessary lint
This commit is contained in:
Nik Clayton
GitHub
parent
a7001eecb8
commit
c154aaa17d
|
|
@ -1701,490 +1701,6 @@
|
||||||
column="57"/>
|
column="57"/>
|
||||||
</issue>
|
</issue>
|
||||||
|
|
||||||
<issue
|
|
||||||
id="LogConditional"
|
|
||||||
message="The log call Log.d(...) should be conditional: surround with `if (Log.isLoggable(...))` or `if (BuildConfig.DEBUG) { ... }`"
|
|
||||||
errorLine1=" Log.d(TAG, "addAccount: saving account with id " + it.id)"
|
|
||||||
errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~">
|
|
||||||
<location
|
|
||||||
file="src/main/java/com/keylesspalace/tusky/db/AccountManager.kt"
|
|
||||||
line="71"
|
|
||||||
column="13"/>
|
|
||||||
</issue>
|
|
||||||
|
|
||||||
<issue
|
|
||||||
id="LogConditional"
|
|
||||||
message="The log call Log.d(...) should be conditional: surround with `if (Log.isLoggable(...))` or `if (BuildConfig.DEBUG) { ... }`"
|
|
||||||
errorLine1=" Log.d(TAG, "updateActiveAccount: saving account with id " + it.id)"
|
|
||||||
errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~">
|
|
||||||
<location
|
|
||||||
file="src/main/java/com/keylesspalace/tusky/db/AccountManager.kt"
|
|
||||||
line="158"
|
|
||||||
column="13"/>
|
|
||||||
</issue>
|
|
||||||
|
|
||||||
<issue
|
|
||||||
id="LogConditional"
|
|
||||||
message="The log call Log.d(...) should be conditional: surround with `if (Log.isLoggable(...))` or `if (BuildConfig.DEBUG) { ... }`"
|
|
||||||
errorLine1=" Log.d(TAG, "setActiveAccount: saving account with id " + it.id)"
|
|
||||||
errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~">
|
|
||||||
<location
|
|
||||||
file="src/main/java/com/keylesspalace/tusky/db/AccountManager.kt"
|
|
||||||
line="173"
|
|
||||||
column="13"/>
|
|
||||||
</issue>
|
|
||||||
|
|
||||||
<issue
|
|
||||||
id="LogConditional"
|
|
||||||
message="The log call Log.i(...) should be conditional: surround with `if (Log.isLoggable(...))` or `if (BuildConfig.DEBUG) { ... }`"
|
|
||||||
errorLine1=" Log.i("
|
|
||||||
errorLine2=" ^">
|
|
||||||
<location
|
|
||||||
file="src/main/java/com/keylesspalace/tusky/viewmodel/AccountsInListViewModel.kt"
|
|
||||||
line="67"
|
|
||||||
column="25"/>
|
|
||||||
</issue>
|
|
||||||
|
|
||||||
<issue
|
|
||||||
id="LogConditional"
|
|
||||||
message="The log call Log.i(...) should be conditional: surround with `if (Log.isLoggable(...))` or `if (BuildConfig.DEBUG) { ... }`"
|
|
||||||
errorLine1=" Log.i("
|
|
||||||
errorLine2=" ^">
|
|
||||||
<location
|
|
||||||
file="src/main/java/com/keylesspalace/tusky/viewmodel/AccountsInListViewModel.kt"
|
|
||||||
line="90"
|
|
||||||
column="25"/>
|
|
||||||
</issue>
|
|
||||||
|
|
||||||
<issue
|
|
||||||
id="LogConditional"
|
|
||||||
message="The log call Log.v(...) should be conditional: surround with `if (Log.isLoggable(...))` or `if (BuildConfig.DEBUG) { ... }`"
|
|
||||||
errorLine1=" clickedSpan?.let { Log.v(TAG, "padding click: ${(clickedSpan as URLSpan).url}") }"
|
|
||||||
errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~">
|
|
||||||
<location
|
|
||||||
file="src/main/java/com/keylesspalace/tusky/view/ClickableSpanTextView.kt"
|
|
||||||
line="232"
|
|
||||||
column="36"/>
|
|
||||||
</issue>
|
|
||||||
|
|
||||||
<issue
|
|
||||||
id="LogConditional"
|
|
||||||
message="The log call Log.v(...) should be conditional: surround with `if (Log.isLoggable(...))` or `if (BuildConfig.DEBUG) { ... }`"
|
|
||||||
errorLine1=" Log.v(TAG, "first: $first second: $second click: $x $y")"
|
|
||||||
errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~">
|
|
||||||
<location
|
|
||||||
file="src/main/java/com/keylesspalace/tusky/view/ClickableSpanTextView.kt"
|
|
||||||
line="330"
|
|
||||||
column="9"/>
|
|
||||||
</issue>
|
|
||||||
|
|
||||||
<issue
|
|
||||||
id="LogConditional"
|
|
||||||
message="The log call Log.d(...) should be conditional: surround with `if (Log.isLoggable(...))` or `if (BuildConfig.DEBUG) { ... }`"
|
|
||||||
errorLine1=" Log.d(TAG, "firstDiff: $firstDiff secondDiff: $secondDiff")"
|
|
||||||
errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~">
|
|
||||||
<location
|
|
||||||
file="src/main/java/com/keylesspalace/tusky/view/ClickableSpanTextView.kt"
|
|
||||||
line="338"
|
|
||||||
column="9"/>
|
|
||||||
</issue>
|
|
||||||
|
|
||||||
<issue
|
|
||||||
id="LogConditional"
|
|
||||||
message="The log call Log.d(...) should be conditional: surround with `if (Log.isLoggable(...))` or `if (BuildConfig.DEBUG) { ... }`"
|
|
||||||
errorLine1=" Log.d("
|
|
||||||
errorLine2=" ^">
|
|
||||||
<location
|
|
||||||
file="src/main/java/com/keylesspalace/tusky/usecase/DeveloperToolsUseCase.kt"
|
|
||||||
line="33"
|
|
||||||
column="13"/>
|
|
||||||
</issue>
|
|
||||||
|
|
||||||
<issue
|
|
||||||
id="LogConditional"
|
|
||||||
message="The log call Log.d(...) should be conditional: surround with `if (Log.isLoggable(...))` or `if (BuildConfig.DEBUG) { ... }`"
|
|
||||||
errorLine1=" Log.d(TAG, "User id $activeAccountId changed: Notification-worthy draft count $count")"
|
|
||||||
errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~">
|
|
||||||
<location
|
|
||||||
file="src/main/java/com/keylesspalace/tusky/db/DraftsAlert.kt"
|
|
||||||
line="61"
|
|
||||||
column="21"/>
|
|
||||||
</issue>
|
|
||||||
|
|
||||||
<issue
|
|
||||||
id="LogConditional"
|
|
||||||
message="The log call Log.d(...) should be conditional: surround with `if (Log.isLoggable(...))` or `if (BuildConfig.DEBUG) { ... }`"
|
|
||||||
errorLine1=" Log.d(TAG, "User id $activeAccountId: Clean out notification-worthy drafts")"
|
|
||||||
errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~">
|
|
||||||
<location
|
|
||||||
file="src/main/java/com/keylesspalace/tusky/db/DraftsAlert.kt"
|
|
||||||
line="82"
|
|
||||||
column="21"/>
|
|
||||||
</issue>
|
|
||||||
|
|
||||||
<issue
|
|
||||||
id="LogConditional"
|
|
||||||
message="The log call Log.d(...) should be conditional: surround with `if (Log.isLoggable(...))` or `if (BuildConfig.DEBUG) { ... }`"
|
|
||||||
errorLine1=" Log.d("LoginWeb", "Failed to load ${data.url}: $error")"
|
|
||||||
errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~">
|
|
||||||
<location
|
|
||||||
file="src/main/java/com/keylesspalace/tusky/components/login/LoginWebViewActivity.kt"
|
|
||||||
line="151"
|
|
||||||
column="17"/>
|
|
||||||
</issue>
|
|
||||||
|
|
||||||
<issue
|
|
||||||
id="LogConditional"
|
|
||||||
message="The log call Log.d(...) should be conditional: surround with `if (Log.isLoggable(...))` or `if (BuildConfig.DEBUG) { ... }`"
|
|
||||||
errorLine1=" Log.d("LoginWeb", "Failed to load ${data.url}: $error")"
|
|
||||||
errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~">
|
|
||||||
<location
|
|
||||||
file="src/main/java/com/keylesspalace/tusky/components/login/LoginWebViewActivity.kt"
|
|
||||||
line="151"
|
|
||||||
column="17"/>
|
|
||||||
</issue>
|
|
||||||
|
|
||||||
<issue
|
|
||||||
id="LogConditional"
|
|
||||||
message="The log call Log.d(...) should be conditional: surround with `if (Log.isLoggable(...))` or `if (BuildConfig.DEBUG) { ... }`"
|
|
||||||
errorLine1=" Log.d(TAG, "Developer tools: $which")"
|
|
||||||
errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~">
|
|
||||||
<location
|
|
||||||
file="src/main/java/com/keylesspalace/tusky/MainActivity.kt"
|
|
||||||
line="662"
|
|
||||||
column="17"/>
|
|
||||||
</issue>
|
|
||||||
|
|
||||||
<issue
|
|
||||||
id="LogConditional"
|
|
||||||
message="The log call Log.d(...) should be conditional: surround with `if (Log.isLoggable(...))` or `if (BuildConfig.DEBUG) { ... }`"
|
|
||||||
errorLine1=" Log.d(TAG, "getting Notifications for " + account.fullName)"
|
|
||||||
errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~">
|
|
||||||
<location
|
|
||||||
file="src/main/java/com/keylesspalace/tusky/components/notifications/NotificationFetcher.kt"
|
|
||||||
line="41"
|
|
||||||
column="9"/>
|
|
||||||
</issue>
|
|
||||||
|
|
||||||
<issue
|
|
||||||
id="LogConditional"
|
|
||||||
message="The log call Log.d(...) should be conditional: surround with `if (Log.isLoggable(...))` or `if (BuildConfig.DEBUG) { ... }`"
|
|
||||||
errorLine1=" Log.d(TAG, "Fetched marker: $notificationMarker")"
|
|
||||||
errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~">
|
|
||||||
<location
|
|
||||||
file="src/main/java/com/keylesspalace/tusky/components/notifications/NotificationFetcher.kt"
|
|
||||||
line="72"
|
|
||||||
column="13"/>
|
|
||||||
</issue>
|
|
||||||
|
|
||||||
<issue
|
|
||||||
id="LogConditional"
|
|
||||||
message="The log call Log.d(...) should be conditional: surround with `if (Log.isLoggable(...))` or `if (BuildConfig.DEBUG) { ... }`"
|
|
||||||
errorLine1=" Log.d(TAG, Log.getStackTraceString(e));"
|
|
||||||
errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~">
|
|
||||||
<location
|
|
||||||
file="src/main/java/com/keylesspalace/tusky/components/notifications/NotificationHelper.java"
|
|
||||||
line="162"
|
|
||||||
column="17"/>
|
|
||||||
</issue>
|
|
||||||
|
|
||||||
<issue
|
|
||||||
id="LogConditional"
|
|
||||||
message="The log call Log.d(...) should be conditional: surround with `if (Log.isLoggable(...))` or `if (BuildConfig.DEBUG) { ... }`"
|
|
||||||
errorLine1=" Log.d(TAG, error.toString())"
|
|
||||||
errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~">
|
|
||||||
<location
|
|
||||||
file="src/main/java/com/keylesspalace/tusky/components/notifications/NotificationsFragment.kt"
|
|
||||||
line="251"
|
|
||||||
column="25"/>
|
|
||||||
</issue>
|
|
||||||
|
|
||||||
<issue
|
|
||||||
id="LogConditional"
|
|
||||||
message="The log call Log.d(...) should be conditional: surround with `if (Log.isLoggable(...))` or `if (BuildConfig.DEBUG) { ... }`"
|
|
||||||
errorLine1=" Log.d(TAG, "load() with ${params.javaClass.simpleName} for key: ${params.key}")"
|
|
||||||
errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~">
|
|
||||||
<location
|
|
||||||
file="src/main/java/com/keylesspalace/tusky/components/notifications/NotificationsPagingSource.kt"
|
|
||||||
line="43"
|
|
||||||
column="9"/>
|
|
||||||
</issue>
|
|
||||||
|
|
||||||
<issue
|
|
||||||
id="LogConditional"
|
|
||||||
message="The log call Log.d(...) should be conditional: surround with `if (Log.isLoggable(...))` or `if (BuildConfig.DEBUG) { ... }`"
|
|
||||||
errorLine1=" Log.d(TAG, "getNotificationsStream(), filtering: $filter")"
|
|
||||||
errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~">
|
|
||||||
<location
|
|
||||||
file="src/main/java/com/keylesspalace/tusky/components/notifications/NotificationsRepository.kt"
|
|
||||||
line="49"
|
|
||||||
column="9"/>
|
|
||||||
</issue>
|
|
||||||
|
|
||||||
<issue
|
|
||||||
id="LogConditional"
|
|
||||||
message="The log call Log.d(...) should be conditional: surround with `if (Log.isLoggable(...))` or `if (BuildConfig.DEBUG) { ... }`"
|
|
||||||
errorLine1=" Log.d(TAG, "notificationFilter: $action")"
|
|
||||||
errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~">
|
|
||||||
<location
|
|
||||||
file="src/main/java/com/keylesspalace/tusky/components/notifications/NotificationsViewModel.kt"
|
|
||||||
line="318"
|
|
||||||
column="17"/>
|
|
||||||
</issue>
|
|
||||||
|
|
||||||
<issue
|
|
||||||
id="LogConditional"
|
|
||||||
message="The log call Log.d(...) should be conditional: surround with `if (Log.isLoggable(...))` or `if (BuildConfig.DEBUG) { ... }`"
|
|
||||||
errorLine1=" Log.d(TAG, "Saving visible ID: ${action.visibleId}")"
|
|
||||||
errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~">
|
|
||||||
<location
|
|
||||||
file="src/main/java/com/keylesspalace/tusky/components/notifications/NotificationsViewModel.kt"
|
|
||||||
line="339"
|
|
||||||
column="21"/>
|
|
||||||
</issue>
|
|
||||||
|
|
||||||
<issue
|
|
||||||
id="LogConditional"
|
|
||||||
message="The log call Log.d(...) should be conditional: surround with `if (Log.isLoggable(...))` or `if (BuildConfig.DEBUG) { ... }`"
|
|
||||||
errorLine1=" Log.d(TAG, "Restoring at $lastNotificationId")"
|
|
||||||
errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~">
|
|
||||||
<location
|
|
||||||
file="src/main/java/com/keylesspalace/tusky/components/notifications/NotificationsViewModel.kt"
|
|
||||||
line="464"
|
|
||||||
column="9"/>
|
|
||||||
</issue>
|
|
||||||
|
|
||||||
<issue
|
|
||||||
id="LogConditional"
|
|
||||||
message="The log call Log.d(...) should be conditional: surround with `if (Log.isLoggable(...))` or `if (BuildConfig.DEBUG) { ... }`"
|
|
||||||
errorLine1=" Log.d(TAG, "UnifiedPush registration succeeded for account ${account.id}")"
|
|
||||||
errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~">
|
|
||||||
<location
|
|
||||||
file="src/main/java/com/keylesspalace/tusky/components/notifications/PushNotificationHelper.kt"
|
|
||||||
line="186"
|
|
||||||
column="9"/>
|
|
||||||
</issue>
|
|
||||||
|
|
||||||
<issue
|
|
||||||
id="LogConditional"
|
|
||||||
message="The log call Log.d(...) should be conditional: surround with `if (Log.isLoggable(...))` or `if (BuildConfig.DEBUG) { ... }`"
|
|
||||||
errorLine1=" Log.d(TAG, "UnifiedPush subscription updated for account ${account.id}")"
|
|
||||||
errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~">
|
|
||||||
<location
|
|
||||||
file="src/main/java/com/keylesspalace/tusky/components/notifications/PushNotificationHelper.kt"
|
|
||||||
line="205"
|
|
||||||
column="13"/>
|
|
||||||
</issue>
|
|
||||||
|
|
||||||
<issue
|
|
||||||
id="LogConditional"
|
|
||||||
message="The log call Log.d(...) should be conditional: surround with `if (Log.isLoggable(...))` or `if (BuildConfig.DEBUG) { ... }`"
|
|
||||||
errorLine1=" Log.d(TAG, "UnifiedPush unregistration succeeded for account " + account.id)"
|
|
||||||
errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~">
|
|
||||||
<location
|
|
||||||
file="src/main/java/com/keylesspalace/tusky/components/notifications/PushNotificationHelper.kt"
|
|
||||||
line="220"
|
|
||||||
column="17"/>
|
|
||||||
</issue>
|
|
||||||
|
|
||||||
<issue
|
|
||||||
id="LogConditional"
|
|
||||||
message="The log call Log.d(...) should be conditional: surround with `if (Log.isLoggable(...))` or `if (BuildConfig.DEBUG) { ... }`"
|
|
||||||
errorLine1=" Log.d(TAG, "Failed to reblog status ${statusViewData.id}", t)"
|
|
||||||
errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~">
|
|
||||||
<location
|
|
||||||
file="src/main/java/com/keylesspalace/tusky/components/search/SearchViewModel.kt"
|
|
||||||
line="128"
|
|
||||||
column="17"/>
|
|
||||||
</issue>
|
|
||||||
|
|
||||||
<issue
|
|
||||||
id="LogConditional"
|
|
||||||
message="The log call Log.d(...) should be conditional: surround with `if (Log.isLoggable(...))` or `if (BuildConfig.DEBUG) { ... }`"
|
|
||||||
errorLine1=" .onFailure { t -> Log.d(TAG, "Failed to vote in poll: ${statusViewData.id}", t) }"
|
|
||||||
errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~">
|
|
||||||
<location
|
|
||||||
file="src/main/java/com/keylesspalace/tusky/components/search/SearchViewModel.kt"
|
|
||||||
line="146"
|
|
||||||
column="35"/>
|
|
||||||
</issue>
|
|
||||||
|
|
||||||
<issue
|
|
||||||
id="LogConditional"
|
|
||||||
message="The log call Log.d(...) should be conditional: surround with `if (Log.isLoggable(...))` or `if (BuildConfig.DEBUG) { ... }`"
|
|
||||||
errorLine1=" Log.d(TAG, "talkback was enabled: $wasEnabled, now $talkBackWasEnabled")"
|
|
||||||
errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~">
|
|
||||||
<location
|
|
||||||
file="src/main/java/com/keylesspalace/tusky/components/timeline/TimelineFragment.kt"
|
|
||||||
line="588"
|
|
||||||
column="9"/>
|
|
||||||
</issue>
|
|
||||||
|
|
||||||
<issue
|
|
||||||
id="LogConditional"
|
|
||||||
message="The log call Log.d(...) should be conditional: surround with `if (Log.isLoggable(...))` or `if (BuildConfig.DEBUG) { ... }`"
|
|
||||||
errorLine1=" Log.d(TAG, "Failed to reblog status " + status.actionableId, t)"
|
|
||||||
errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~">
|
|
||||||
<location
|
|
||||||
file="src/main/java/com/keylesspalace/tusky/components/timeline/viewmodel/TimelineViewModel.kt"
|
|
||||||
line="114"
|
|
||||||
column="17"/>
|
|
||||||
</issue>
|
|
||||||
|
|
||||||
<issue
|
|
||||||
id="LogConditional"
|
|
||||||
message="The log call Log.d(...) should be conditional: surround with `if (Log.isLoggable(...))` or `if (BuildConfig.DEBUG) { ... }`"
|
|
||||||
errorLine1=" Log.d(TAG, "Failed to favourite status " + status.actionableId, t)"
|
|
||||||
errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~">
|
|
||||||
<location
|
|
||||||
file="src/main/java/com/keylesspalace/tusky/components/timeline/viewmodel/TimelineViewModel.kt"
|
|
||||||
line="124"
|
|
||||||
column="17"/>
|
|
||||||
</issue>
|
|
||||||
|
|
||||||
<issue
|
|
||||||
id="LogConditional"
|
|
||||||
message="The log call Log.d(...) should be conditional: surround with `if (Log.isLoggable(...))` or `if (BuildConfig.DEBUG) { ... }`"
|
|
||||||
errorLine1=" Log.d(TAG, "Failed to bookmark status " + status.actionableId, t)"
|
|
||||||
errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~">
|
|
||||||
<location
|
|
||||||
file="src/main/java/com/keylesspalace/tusky/components/timeline/viewmodel/TimelineViewModel.kt"
|
|
||||||
line="134"
|
|
||||||
column="17"/>
|
|
||||||
</issue>
|
|
||||||
|
|
||||||
<issue
|
|
||||||
id="LogConditional"
|
|
||||||
message="The log call Log.d(...) should be conditional: surround with `if (Log.isLoggable(...))` or `if (BuildConfig.DEBUG) { ... }`"
|
|
||||||
errorLine1=" Log.d(TAG, "Failed to vote in poll: " + status.actionableId, t)"
|
|
||||||
errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~">
|
|
||||||
<location
|
|
||||||
file="src/main/java/com/keylesspalace/tusky/components/timeline/viewmodel/TimelineViewModel.kt"
|
|
||||||
line="152"
|
|
||||||
column="17"/>
|
|
||||||
</issue>
|
|
||||||
|
|
||||||
<issue
|
|
||||||
id="LogConditional"
|
|
||||||
message="The log call Log.d(...) should be conditional: surround with `if (Log.isLoggable(...))` or `if (BuildConfig.DEBUG) { ... }`"
|
|
||||||
errorLine1=" Log.d(TAG, "talkback was enabled: $wasEnabled, now $talkBackWasEnabled")"
|
|
||||||
errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~">
|
|
||||||
<location
|
|
||||||
file="src/main/java/com/keylesspalace/tusky/components/trending/TrendingFragment.kt"
|
|
||||||
line="275"
|
|
||||||
column="9"/>
|
|
||||||
</issue>
|
|
||||||
|
|
||||||
<issue
|
|
||||||
id="LogConditional"
|
|
||||||
message="The log call Log.d(...) should be conditional: surround with `if (Log.isLoggable(...))` or `if (BuildConfig.DEBUG) { ... }`"
|
|
||||||
errorLine1=" Log.d(TAG, "Upgrading shared preferences: $oldVersion -> $newVersion")"
|
|
||||||
errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~">
|
|
||||||
<location
|
|
||||||
file="src/main/java/com/keylesspalace/tusky/TuskyApplication.kt"
|
|
||||||
line="104"
|
|
||||||
column="9"/>
|
|
||||||
</issue>
|
|
||||||
|
|
||||||
<issue
|
|
||||||
id="LogConditional"
|
|
||||||
message="The log call Log.d(...) should be conditional: surround with `if (Log.isLoggable(...))` or `if (BuildConfig.DEBUG) { ... }`"
|
|
||||||
errorLine1=" Log.d(TAG, "New message received for account $instance")"
|
|
||||||
errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~">
|
|
||||||
<location
|
|
||||||
file="src/main/java/com/keylesspalace/tusky/receiver/UnifiedPushBroadcastReceiver.kt"
|
|
||||||
line="54"
|
|
||||||
column="9"/>
|
|
||||||
</issue>
|
|
||||||
|
|
||||||
<issue
|
|
||||||
id="LogConditional"
|
|
||||||
message="The log call Log.d(...) should be conditional: surround with `if (Log.isLoggable(...))` or `if (BuildConfig.DEBUG) { ... }`"
|
|
||||||
errorLine1=" Log.d(TAG, "Endpoint available for account $instance: $endpoint")"
|
|
||||||
errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~">
|
|
||||||
<location
|
|
||||||
file="src/main/java/com/keylesspalace/tusky/receiver/UnifiedPushBroadcastReceiver.kt"
|
|
||||||
line="62"
|
|
||||||
column="9"/>
|
|
||||||
</issue>
|
|
||||||
|
|
||||||
<issue
|
|
||||||
id="LogConditional"
|
|
||||||
message="The log call Log.d(...) should be conditional: surround with `if (Log.isLoggable(...))` or `if (BuildConfig.DEBUG) { ... }`"
|
|
||||||
errorLine1=" Log.d(TAG, "Endpoint unregistered for account $instance")"
|
|
||||||
errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~">
|
|
||||||
<location
|
|
||||||
file="src/main/java/com/keylesspalace/tusky/receiver/UnifiedPushBroadcastReceiver.kt"
|
|
||||||
line="74"
|
|
||||||
column="9"/>
|
|
||||||
</issue>
|
|
||||||
|
|
||||||
<issue
|
|
||||||
id="LogConditional"
|
|
||||||
message="The log call Log.d(...) should be conditional: surround with `if (Log.isLoggable(...))` or `if (BuildConfig.DEBUG) { ... }`"
|
|
||||||
errorLine1=" Log.d(TAG, "Download image result: $result")"
|
|
||||||
errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~">
|
|
||||||
<location
|
|
||||||
file="src/main/java/com/keylesspalace/tusky/ViewMediaActivity.kt"
|
|
||||||
line="305"
|
|
||||||
column="21"/>
|
|
||||||
</issue>
|
|
||||||
|
|
||||||
<issue
|
|
||||||
id="LogConditional"
|
|
||||||
message="The log call Log.d(...) should be conditional: surround with `if (Log.isLoggable(...))` or `if (BuildConfig.DEBUG) { ... }`"
|
|
||||||
errorLine1=" Log.d(TAG, "Finding status with: $id")"
|
|
||||||
errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~">
|
|
||||||
<location
|
|
||||||
file="src/main/java/com/keylesspalace/tusky/components/viewthread/ViewThreadViewModel.kt"
|
|
||||||
line="108"
|
|
||||||
column="13"/>
|
|
||||||
</issue>
|
|
||||||
|
|
||||||
<issue
|
|
||||||
id="LogConditional"
|
|
||||||
message="The log call Log.d(...) should be conditional: surround with `if (Log.isLoggable(...))` or `if (BuildConfig.DEBUG) { ... }`"
|
|
||||||
errorLine1=" Log.d(TAG, "Failed to reblog status " + status.actionableId, t)"
|
|
||||||
errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~">
|
|
||||||
<location
|
|
||||||
file="src/main/java/com/keylesspalace/tusky/components/viewthread/ViewThreadViewModel.kt"
|
|
||||||
line="199"
|
|
||||||
column="17"/>
|
|
||||||
</issue>
|
|
||||||
|
|
||||||
<issue
|
|
||||||
id="LogConditional"
|
|
||||||
message="The log call Log.d(...) should be conditional: surround with `if (Log.isLoggable(...))` or `if (BuildConfig.DEBUG) { ... }`"
|
|
||||||
errorLine1=" Log.d(TAG, "Failed to favourite status " + status.actionableId, t)"
|
|
||||||
errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~">
|
|
||||||
<location
|
|
||||||
file="src/main/java/com/keylesspalace/tusky/components/viewthread/ViewThreadViewModel.kt"
|
|
||||||
line="209"
|
|
||||||
column="17"/>
|
|
||||||
</issue>
|
|
||||||
|
|
||||||
<issue
|
|
||||||
id="LogConditional"
|
|
||||||
message="The log call Log.d(...) should be conditional: surround with `if (Log.isLoggable(...))` or `if (BuildConfig.DEBUG) { ... }`"
|
|
||||||
errorLine1=" Log.d(TAG, "Failed to bookmark status " + status.actionableId, t)"
|
|
||||||
errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~">
|
|
||||||
<location
|
|
||||||
file="src/main/java/com/keylesspalace/tusky/components/viewthread/ViewThreadViewModel.kt"
|
|
||||||
line="219"
|
|
||||||
column="17"/>
|
|
||||||
</issue>
|
|
||||||
|
|
||||||
<issue
|
|
||||||
id="LogConditional"
|
|
||||||
message="The log call Log.d(...) should be conditional: surround with `if (Log.isLoggable(...))` or `if (BuildConfig.DEBUG) { ... }`"
|
|
||||||
errorLine1=" Log.d(TAG, "Failed to vote in poll: " + status.actionableId, t)"
|
|
||||||
errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~">
|
|
||||||
<location
|
|
||||||
file="src/main/java/com/keylesspalace/tusky/components/viewthread/ViewThreadViewModel.kt"
|
|
||||||
line="239"
|
|
||||||
column="17"/>
|
|
||||||
</issue>
|
|
||||||
|
|
||||||
<issue
|
<issue
|
||||||
id="VectorPath"
|
id="VectorPath"
|
||||||
message="Very long vector path (802 characters), which is bad for performance. Considering reducing precision, removing minor details or rasterizing vector."
|
message="Very long vector path (802 characters), which is bad for performance. Considering reducing precision, removing minor details or rasterizing vector."
|
||||||
|
|
|
||||||
|
|
@ -30,6 +30,9 @@
|
||||||
Disable these for the time being. -->
|
Disable these for the time being. -->
|
||||||
<issue id="UnusedIds" severity="ignore" />
|
<issue id="UnusedIds" severity="ignore" />
|
||||||
|
|
||||||
|
<!-- Logs are stripped in release builds. -->
|
||||||
|
<issue id="LogConditional" severity="ignore" />
|
||||||
|
|
||||||
<!-- Ensure we are warned about errors in the baseline -->
|
<!-- Ensure we are warned about errors in the baseline -->
|
||||||
<issue id="LintBaseline" severity="warning" />
|
<issue id="LintBaseline" severity="warning" />
|
||||||
|
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue