From 30be3ce1f0d59692252a58a939f64b0a1ecc7039 Mon Sep 17 00:00:00 2001 From: Nik Clayton Date: Sun, 30 Jul 2023 21:45:43 +0200 Subject: [PATCH] Use lint version 8.1.0 to fix erroneous warning about forEach (#3903) Android lint was erroneously warning that the forEach construct in Kotlin required API 24+, which is incorrect, see https://issuetracker.google.com/issues/185418482. Work around that by forcing the Android lint version to 8.1.0. This triggered some additional checks, which have been ignored, and a new baseline. --- app/lint-baseline.xml | 1052 ++++++----------------------------------- app/lint.xml | 6 + gradle.properties | 3 + 3 files changed, 150 insertions(+), 911 deletions(-) diff --git a/app/lint-baseline.xml b/app/lint-baseline.xml index 056acea99..1dc283511 100644 --- a/app/lint-baseline.xml +++ b/app/lint-baseline.xml @@ -1,5 +1,5 @@ - + @@ -30,7 +30,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"> @@ -41,7 +41,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"> @@ -84,6 +84,17 @@ column="21"/> + + + + @@ -788,6 +799,17 @@ column="293"/> + + + + @@ -828,7 +850,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"> @@ -839,7 +861,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"> @@ -850,7 +872,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"> @@ -861,535 +883,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"> - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - @@ -2148,7 +1642,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~"> @@ -2159,7 +1653,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~"> @@ -2313,7 +1807,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"> @@ -2324,7 +1818,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"> @@ -2335,7 +1829,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~"> @@ -2346,7 +1840,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~"> @@ -2357,7 +1851,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~"> @@ -2368,7 +1862,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"> @@ -2379,7 +1873,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~"> @@ -2390,7 +1884,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"> @@ -2401,7 +1895,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"> @@ -2412,7 +1906,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"> @@ -2423,7 +1917,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"> @@ -2434,7 +1928,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~"> @@ -2445,7 +1939,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"> @@ -2456,7 +1950,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~~~"> @@ -2467,7 +1961,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~"> @@ -2478,7 +1972,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~~"> @@ -2489,7 +1983,7 @@ errorLine2=" ~~~~~~~~~~~~"> @@ -2500,7 +1994,7 @@ errorLine2=" ~~~~~~~~~~~~~~"> @@ -2511,7 +2005,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~"> @@ -2522,7 +2016,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~"> @@ -2533,7 +2027,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~"> @@ -2544,7 +2038,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~"> @@ -2555,7 +2049,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~"> @@ -2566,7 +2060,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~"> @@ -2577,7 +2071,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"> @@ -2588,7 +2082,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~"> @@ -2599,7 +2093,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"> @@ -2610,7 +2104,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"> @@ -2621,7 +2115,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"> @@ -2632,7 +2126,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~~"> @@ -2643,7 +2137,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~"> @@ -2654,7 +2148,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~"> @@ -2665,7 +2159,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"> @@ -2676,7 +2170,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~"> @@ -2863,7 +2357,7 @@ errorLine2=" ~~~~~~~~~~~~~"> @@ -3050,7 +2544,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~"> @@ -3061,7 +2555,7 @@ errorLine2=" ~~~~~~~~~"> @@ -3072,7 +2566,7 @@ errorLine2=" ~~~~~~~~~"> @@ -3644,7 +3138,7 @@ errorLine2=" ~~~~~~~~~~~~"> @@ -3655,7 +3149,7 @@ errorLine2=" ~~~~~~~~~~~~~"> @@ -3666,7 +3160,7 @@ errorLine2=" ~~~~~~~"> @@ -3677,7 +3171,7 @@ errorLine2=" ~~~~~~~"> @@ -3688,7 +3182,7 @@ errorLine2=" ~~~~~~~"> @@ -3699,7 +3193,7 @@ errorLine2=" ~~~~~~~"> @@ -3710,7 +3204,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~"> @@ -3721,7 +3215,7 @@ errorLine2=" ~~~~~~~"> @@ -3732,7 +3226,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~"> @@ -3743,7 +3237,7 @@ errorLine2=" ~~~~~~~"> @@ -3754,7 +3248,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~"> @@ -3765,7 +3259,7 @@ errorLine2=" ~~~~~~~"> @@ -3776,7 +3270,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~"> @@ -3787,7 +3281,7 @@ errorLine2=" ~~~~~~~"> @@ -3798,7 +3292,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~"> @@ -3809,7 +3303,7 @@ errorLine2=" ~~~~~~~"> @@ -3820,7 +3314,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~"> @@ -3831,7 +3325,7 @@ errorLine2=" ~~~~~~~"> @@ -3842,7 +3336,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~"> @@ -3853,7 +3347,7 @@ errorLine2=" ~~~~~~~"> @@ -3864,7 +3358,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~"> @@ -3875,7 +3369,7 @@ errorLine2=" ~~~~~~~"> @@ -3886,7 +3380,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~"> @@ -3897,7 +3391,7 @@ errorLine2=" ~~~~~~~"> @@ -3908,7 +3402,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~"> @@ -3919,7 +3413,7 @@ errorLine2=" ~~~~~~~"> @@ -3930,7 +3424,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~"> @@ -3941,7 +3435,7 @@ errorLine2=" ~~~~~~~"> @@ -3952,7 +3446,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~"> @@ -3963,7 +3457,7 @@ errorLine2=" ~~~~~~~"> @@ -3974,7 +3468,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~"> @@ -3985,7 +3479,7 @@ errorLine2=" ~~~~~~~"> @@ -3996,7 +3490,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~"> @@ -4007,7 +3501,7 @@ errorLine2=" ~~~~~~~"> @@ -4018,7 +3512,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~"> @@ -4029,7 +3523,7 @@ errorLine2=" ~~~~~~~"> @@ -4038,6 +3532,17 @@ message="Access to `private` method `getBinding` of class `MainActivity` requires synthetic accessor" errorLine1=" binding.mainToolbar.title = tab.contentDescription" errorLine2=" ~~~~~~~"> + + + + - - - - @@ -4073,7 +3567,7 @@ errorLine2=" ~~~~~~~"> @@ -4084,7 +3578,7 @@ errorLine2=" ~~~~~~~"> @@ -4095,7 +3589,7 @@ errorLine2=" ~~~~~~~"> @@ -4106,7 +3600,7 @@ errorLine2=" ~~~~~~~"> @@ -4117,7 +3611,7 @@ errorLine2=" ~~~~~~~"> @@ -4128,7 +3622,7 @@ errorLine2=" ~~~~~~~"> @@ -4425,7 +3919,7 @@ errorLine2=" ~~~~~~~~~"> @@ -4436,7 +3930,7 @@ errorLine2=" ~~~~~~~~"> @@ -4447,7 +3941,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~~~"> @@ -4458,7 +3952,7 @@ errorLine2=" ~~~~~~~~~~~~~~~~~~~~~"> @@ -4759,281 +4253,6 @@ column="54"/> - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - @@ -6339,6 +5558,17 @@ column="6"/> + + + + + + + + + + diff --git a/gradle.properties b/gradle.properties index 84a07ffd3..dc6a0c69a 100644 --- a/gradle.properties +++ b/gradle.properties @@ -12,3 +12,6 @@ kotlin.incremental.useClasspathSnapshot=true android.defaults.buildfeatures.resvalues=false android.defaults.buildfeatures.shaders=false android.useAndroidX=true + +# Upgrade lint to a newer version to work around https://issuetracker.google.com/issues/185418482. +android.experimental.lint.version=8.1.0